Skip to content

Commit 413d3cf

Browse files
fsoikinfsoikin
authored
Replace ✅ ❌ ⚠️ emoji with ✓✘ ✘ ‼ espectivelyy (#1258)
1 parent 4e9b5aa commit 413d3cf

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

46 files changed

+112
-111
lines changed

‎CHANGELOG.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ Other improvements:
1818
- `spago repl` now writes a `.purs-repl` file, unless already there, containing `import Prelude`.
1919
- Added typo suggestions upon failing to find a package by name.
2020
- `spago publish` now checks that the publish location matches one of the remotes in the current Git repository.
21+
- Emoji ✅ ❌ ‼️ replaced with ✓ ✘ ‼ respectively, and are not printed at all with `--no-color`.
2122

2223
## [0.21.0] - 2023-05-04
2324

‎core/src/Log.purs‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -128,14 +128,14 @@ logSuccess :: forall a b m. MonadEffect m => MonadAsk (LogEnv b) m => Loggable a
128128
logSuccess l = log
129129
{ level: LogInfo
130130
, content: Ansi.foreground Ansi.Green
131-
(Log.break <> Ansi.bold (toDoc "✅" <> Log.space <> toDoc l) <> Log.break)
131+
(Log.break <> Ansi.bold (toDoc "✓" <> Log.space <> toDoc l) <> Log.break)
132132
}
133133

134134
logFailure :: forall a b m. MonadEffect m => MonadAsk (LogEnv b) m => Loggable a => a -> m Unit
135135
logFailure l = log
136136
{ level: LogInfo
137137
, content: Ansi.foreground Ansi.Red
138-
(Log.break <> Ansi.bold (toDoc "❌" <> Log.space <> toDoc l) <> Log.break)
138+
(Log.break <> Ansi.bold (toDoc "✘" <> Log.space <> toDoc l) <> Log.break)
139139
}
140140

141141
logDebug :: forall a b m. MonadEffect m => MonadAsk (LogEnv b) m => Loggable a => a -> m Unit
@@ -144,7 +144,7 @@ logDebug l = log { level: LogDebug, content: Ansi.foreground Ansi.Blue (toDoc l)
144144
logWarn :: forall a b m. MonadEffect m => MonadAsk (LogEnv b) m => Loggable a => a -> m Unit
145145
logWarn l = log
146146
{ level: LogWarning
147-
, content: Ansi.foreground Ansi.Yellow (Ansi.bold (toDoc "⚠️" <> Log.space <> toDoc l))
147+
, content: Ansi.foreground Ansi.Yellow (Ansi.bold (toDoc "‼" <> Log.space <> toDoc l))
148148
}
149149

150150
logError :: forall a b m. MonadEffect m => MonadAsk (LogEnv b) m => Loggable a => a -> m Unit

‎test-fixtures/alternate-backend-output.txt‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
Reading Spago workspace configuration...
22

3-
✅ Selecting package to build: subpackage
3+
✓ Selecting package to build: subpackage
44

55
Downloading dependencies...
66
Building...
77
Src Lib All
88
Warnings 0 0 0
99
Errors 0 0 0
1010

11-
✅ Build succeeded.
11+
✓ Build succeeded.
1212

1313
Compiling with backend "echo"
1414

15-
✅ Backend build succeeded.
15+
✓ Backend build succeeded.

‎test-fixtures/build/migrate-config/migrated-output.txt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
Reading Spago workspace configuration...
22

3-
✅ Selecting package to build: migrate-config
3+
✓ Selecting package to build: migrate-config
44

55
Downloading dependencies...
66
Building...
77
Src Lib All
88
Warnings 0 0 0
99
Errors 0 0 0
1010

11-
✅ Build succeeded.
11+
✓ Build succeeded.

‎test-fixtures/build/migrate-config/migrating-output.txt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
Reading Spago workspace configuration...
22
Migrating your spago.yaml to the latest version...
33

4-
✅ Selecting package to build: migrate-config
4+
✓ Selecting package to build: migrate-config
55

66
Downloading dependencies...
77
Building...
88
Src Lib All
99
Warnings 0 0 0
1010
Errors 0 0 0
1111

12-
✅ Build succeeded.
12+
✓ Build succeeded.

‎test-fixtures/build/migrate-config/unmigrated-warning.txt‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
Reading Spago workspace configuration...
2-
⚠️ Your spago.yaml is using an outdated format. Run Spago with the --migrate flag to update it to the latest version.
2+
‼ Your spago.yaml is using an outdated format. Run Spago with the --migrate flag to update it to the latest version.
33

4-
✅ Selecting package to build: migrate-config
4+
✓ Selecting package to build: migrate-config
55

66
Downloading dependencies...
77
Building...
88
Src Lib All
99
Warnings 0 0 0
1010
Errors 0 0 0
1111

12-
✅ Build succeeded.
12+
✓ Build succeeded.

‎test-fixtures/censor-stats-output.txt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
Reading Spago workspace configuration...
22

3-
✅ Selecting package to build: aaa
3+
✓ Selecting package to build: aaa
44

55
Downloading dependencies...
66
Building...
77

88

9-
✅ Build succeeded.
9+
✓ Build succeeded.

‎test-fixtures/circular-dependencies.txt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
Reading Spago workspace configuration...
22

3-
✅ Selecting package to build: bbb
3+
✓ Selecting package to build: bbb
44

55
Cloning https://github.com/purescript/spago.git
66
Cloning https://github.com/purescript/spago.git
77

8-
❌ The following packages have circular dependencies:
8+
✘ The following packages have circular dependencies:
99
- a
1010
- b

‎test-fixtures/codegen-opt.txt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
Reading Spago workspace configuration...
22

3-
✅ Selecting package to build: 7368613235362d68766258694c614d517a3667747a58725778
3+
✓ Selecting package to build: 7368613235362d68766258694c614d517a3667747a58725778
44

55
Downloading dependencies...
66
Building...
77

8-
❌ Can't pass the `--codegen` option to purs, Spago already does that for you.
8+
✘ Can't pass the `--codegen` option to purs, Spago already does that for you.
99
Remove the argument to make this error go away!

‎test-fixtures/list-packages-registry.txt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
Reading Spago workspace configuration...
22

3-
✅ Selecting package to build: aaa
3+
✓ Selecting package to build: aaa
44

55
Downloading dependencies...
66
No lockfile found, generating it...
77
Lockfile written to spago.lock. Please commit this file.
88

9-
❌ Cannot list the packages in the package set, as none is configured for the project.
9+
✘ Cannot list the packages in the package set, as none is configured for the project.

0 commit comments

Comments
 (0)