Replace ✅ ❌ ⚠️ emoji with ✓✘ ✘ ‼ espectivelyy (#1258)

purescript · Jul 29, 2024 · 413d3cf · 413d3cf
1 parent 4e9b5aa
commit 413d3cf
Show file tree

Hide file tree

Showing 46 changed files with 112 additions and 111 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -18,6 +18,7 @@ Other improvements:
 - `spago repl` now writes a `.purs-repl` file, unless already there, containing `import Prelude`.
 - Added typo suggestions upon failing to find a package by name.
 - `spago publish` now checks that the publish location matches one of the remotes in the current Git repository.
+- Emoji ✅ ❌ ‼️ replaced with ✓ ✘ ‼ respectively, and are not printed at all with `--no-color`.
 
 ## [0.21.0] - 2023-05-04
 

diff --git a/core/src/Log.purs b/core/src/Log.purs
@@ -128,14 +128,14 @@ logSuccess :: forall a b m. MonadEffect m => MonadAsk (LogEnv b) m => Loggable a
 logSuccess l = log
   { level: LogInfo
   , content: Ansi.foreground Ansi.Green
-      (Log.break <> Ansi.bold (toDoc "✅" <> Log.space <> toDoc l) <> Log.break)
+      (Log.break <> Ansi.bold (toDoc "✓" <> Log.space <> toDoc l) <> Log.break)
   }
 
 logFailure :: forall a b m. MonadEffect m => MonadAsk (LogEnv b) m => Loggable a => a -> m Unit
 logFailure l = log
   { level: LogInfo
   , content: Ansi.foreground Ansi.Red
-      (Log.break <> Ansi.bold (toDoc "❌" <> Log.space <> toDoc l) <> Log.break)
+      (Log.break <> Ansi.bold (toDoc "✘" <> Log.space <> toDoc l) <> Log.break)
   }
 
 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)
 logWarn :: forall a b m. MonadEffect m => MonadAsk (LogEnv b) m => Loggable a => a -> m Unit
 logWarn l = log
   { level: LogWarning
-  , content: Ansi.foreground Ansi.Yellow (Ansi.bold (toDoc "⚠️" <> Log.space <> toDoc l))
+  , content: Ansi.foreground Ansi.Yellow (Ansi.bold (toDoc "‼" <> Log.space <> toDoc l))
   }
 
 logError :: forall a b m. MonadEffect m => MonadAsk (LogEnv b) m => Loggable a => a -> m Unit

diff --git a/test-fixtures/alternate-backend-output.txt b/test-fixtures/alternate-backend-output.txt
@@ -1,15 +1,15 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: subpackage
+✓ Selecting package to build: subpackage
 
 Downloading dependencies...
 Building...
            Src   Lib   All
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
 
 Compiling with backend "echo"
 
-✅ Backend build succeeded.
+✓ Backend build succeeded.
diff --git a/test-fixtures/build/migrate-config/migrated-output.txt b/test-fixtures/build/migrate-config/migrated-output.txt
@@ -1,11 +1,11 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: migrate-config
+✓ Selecting package to build: migrate-config
 
 Downloading dependencies...
 Building...
            Src   Lib   All
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
diff --git a/test-fixtures/build/migrate-config/migrating-output.txt b/test-fixtures/build/migrate-config/migrating-output.txt
@@ -1,12 +1,12 @@
 Reading Spago workspace configuration...
 Migrating your spago.yaml to the latest version...
 
-✅ Selecting package to build: migrate-config
+✓ Selecting package to build: migrate-config
 
 Downloading dependencies...
 Building...
            Src   Lib   All
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
diff --git a/test-fixtures/build/migrate-config/unmigrated-warning.txt b/test-fixtures/build/migrate-config/unmigrated-warning.txt
@@ -1,12 +1,12 @@
 Reading Spago workspace configuration...
-⚠️ Your spago.yaml is using an outdated format. Run Spago with the --migrate flag to update it to the latest version.
+‼ Your spago.yaml is using an outdated format. Run Spago with the --migrate flag to update it to the latest version.
 
-✅ Selecting package to build: migrate-config
+✓ Selecting package to build: migrate-config
 
 Downloading dependencies...
 Building...
            Src   Lib   All
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
diff --git a/test-fixtures/censor-stats-output.txt b/test-fixtures/censor-stats-output.txt
@@ -1,9 +1,9 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: aaa
+✓ Selecting package to build: aaa
 
 Downloading dependencies...
 Building...
 
 
-✅ Build succeeded.
+✓ Build succeeded.
diff --git a/test-fixtures/circular-dependencies.txt b/test-fixtures/circular-dependencies.txt
@@ -1,10 +1,10 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: bbb
+✓ Selecting package to build: bbb
 
 Cloning https://github.com/purescript/spago.git
 Cloning https://github.com/purescript/spago.git
 
-❌ The following packages have circular dependencies:
+✘ The following packages have circular dependencies:
   - a
   - b
diff --git a/test-fixtures/codegen-opt.txt b/test-fixtures/codegen-opt.txt
@@ -1,9 +1,9 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: 7368613235362d68766258694c614d517a3667747a58725778
+✓ Selecting package to build: 7368613235362d68766258694c614d517a3667747a58725778
 
 Downloading dependencies...
 Building...
 
-❌ Can't pass the `--codegen` option to purs, Spago already does that for you.
+✘ Can't pass the `--codegen` option to purs, Spago already does that for you.
 Remove the argument to make this error go away!
diff --git a/test-fixtures/list-packages-registry.txt b/test-fixtures/list-packages-registry.txt
@@ -1,9 +1,9 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: aaa
+✓ Selecting package to build: aaa
 
 Downloading dependencies...
 No lockfile found, generating it...
 Lockfile written to spago.lock. Please commit this file.
 
-❌ Cannot list the packages in the package set, as none is configured for the project.
+✘ Cannot list the packages in the package set, as none is configured for the project.
diff --git a/test-fixtures/missing-dependencies.txt b/test-fixtures/missing-dependencies.txt
@@ -1,9 +1,9 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: aaaa
+✓ Selecting package to build: aaaa
 
 
-❌ The following packages do not exist in your package set:
+✘ The following packages do not exist in your package set:
   - arrys (did you mean: arrays)
   - bar-bar-bar
   - effcet (did you mean: effect)

diff --git a/test-fixtures/monorepo/ignore-nested-workspaces/expected-stderr.txt b/test-fixtures/monorepo/ignore-nested-workspaces/expected-stderr.txt
@@ -1,11 +1,11 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: ignore-nested-workspaces
+✓ Selecting package to build: ignore-nested-workspaces
 
 Downloading dependencies...
 Building...
            Src   Lib   All
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
diff --git a/test-fixtures/monorepo/local-packages-work-with-solver/expected-stderr.txt b/test-fixtures/monorepo/local-packages-work-with-solver/expected-stderr.txt
@@ -1,6 +1,6 @@
 Reading Spago workspace configuration...
 
-✅ Selecting 2 packages to build:
+✓ Selecting 2 packages to build:
     app
     lib
 
@@ -10,4 +10,4 @@ Building...
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
diff --git a/test-fixtures/monorepo/pedantic-cross-package-imports/expected-stderr.txt b/test-fixtures/monorepo/pedantic-cross-package-imports/expected-stderr.txt
@@ -1,6 +1,6 @@
 Reading Spago workspace configuration...
 
-✅ Selecting 2 packages to build:
+✓ Selecting 2 packages to build:
     pedantic-cross-package-imports
     subpackage
 
@@ -10,11 +10,11 @@ Building...
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
 
 Looking for unused and undeclared transitive dependencies...
 
-❌ Found unused and/or undeclared transitive dependencies:
+✘ Found unused and/or undeclared transitive dependencies:
 
 Sources for package 'subpackage' import the following transitive dependencies - please add them to the project dependencies, or remove the imports:
   pedantic-cross-package-imports

diff --git a/test-fixtures/offline.txt b/test-fixtures/offline.txt
@@ -1,6 +1,6 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: eee
+✓ Selecting package to build: eee
 
 
-❌ You are offline and the repo 'https://github.com/purescript/purescript-either.git' is not available locally, can't make progress.
+✘ You are offline and the repo 'https://github.com/purescript/purescript-either.git' is not available locally, can't make progress.
diff --git a/test-fixtures/package-name-too-long-stderr.txt b/test-fixtures/package-name-too-long-stderr.txt
@@ -1,4 +1,4 @@
-❌ Could not figure out a name for the new package. Error:
+✘ Could not figure out a name for the new package. Error:
 
 
     Package name cannot be longer than 150 characters
diff --git a/test-fixtures/package-typo-suggestions/1.txt b/test-fixtures/package-typo-suggestions/1.txt
@@ -1,6 +1,6 @@
 Reading Spago workspace configuration...
 
-❌ Selected package inder was not found in the local packages.
+✘ Selected package inder was not found in the local packages.
 Did you mean:
   finder
   binder
diff --git a/test-fixtures/package-typo-suggestions/2.txt b/test-fixtures/package-typo-suggestions/2.txt
@@ -1,5 +1,5 @@
 Reading Spago workspace configuration...
 
-❌ Selected package flounder was not found in the local packages.
+✘ Selected package flounder was not found in the local packages.
 Did you mean:
   founder
diff --git a/test-fixtures/package-typo-suggestions/3.txt b/test-fixtures/package-typo-suggestions/3.txt
@@ -1,6 +1,6 @@
 Reading Spago workspace configuration...
 
-❌ Selected package totally-bogus was not found in the local packages.
+✘ Selected package totally-bogus was not found in the local packages.
 All available packages:
   binder
   finder

diff --git a/test-fixtures/pedantic/check-direct-import-transitive-dependency-both.txt b/test-fixtures/pedantic/check-direct-import-transitive-dependency-both.txt
@@ -1,18 +1,18 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: pedantic
+✓ Selecting package to build: pedantic
 
 Downloading dependencies...
 Building...
            Src   Lib   All
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
 
 Looking for unused and undeclared transitive dependencies...
 
-❌ Found unused and/or undeclared transitive dependencies:
+✘ Found unused and/or undeclared transitive dependencies:
 
 Sources for package 'pedantic' import the following transitive dependencies - please add them to the project dependencies, or remove the imports:
   maybe

diff --git a/test-fixtures/pedantic/check-direct-import-transitive-dependency-test.txt b/test-fixtures/pedantic/check-direct-import-transitive-dependency-test.txt
@@ -1,18 +1,18 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: pedantic
+✓ Selecting package to build: pedantic
 
 Downloading dependencies...
 Building...
            Src   Lib   All
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
 
 Looking for unused and undeclared transitive dependencies...
 
-❌ Found unused and/or undeclared transitive dependencies:
+✘ Found unused and/or undeclared transitive dependencies:
 
 Tests for package 'pedantic' import the following transitive dependencies - please add them to the project dependencies, or remove the imports:
   control

diff --git a/test-fixtures/pedantic/check-direct-import-transitive-dependency.txt b/test-fixtures/pedantic/check-direct-import-transitive-dependency.txt
@@ -1,18 +1,18 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: pedantic
+✓ Selecting package to build: pedantic
 
 Downloading dependencies...
 Building...
            Src   Lib   All
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
 
 Looking for unused and undeclared transitive dependencies...
 
-❌ Found unused and/or undeclared transitive dependencies:
+✘ Found unused and/or undeclared transitive dependencies:
 
 Sources for package 'pedantic' import the following transitive dependencies - please add them to the project dependencies, or remove the imports:
   control

diff --git a/test-fixtures/pedantic/check-pedantic-packages.txt b/test-fixtures/pedantic/check-pedantic-packages.txt
@@ -1,18 +1,18 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: pedantic
+✓ Selecting package to build: pedantic
 
 Downloading dependencies...
 Building...
            Src   Lib   All
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
 
 Looking for unused and undeclared transitive dependencies...
 
-❌ Found unused and/or undeclared transitive dependencies:
+✘ Found unused and/or undeclared transitive dependencies:
 
 Sources for package 'pedantic' declares unused dependencies - please remove them from the project config:
   - console

diff --git a/test-fixtures/pedantic/check-unused-dependency-in-source.txt b/test-fixtures/pedantic/check-unused-dependency-in-source.txt
@@ -1,18 +1,18 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: pedantic
+✓ Selecting package to build: pedantic
 
 Downloading dependencies...
 Building...
            Src   Lib   All
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
 
 Looking for unused and undeclared transitive dependencies...
 
-❌ Found unused and/or undeclared transitive dependencies:
+✘ Found unused and/or undeclared transitive dependencies:
 
 Sources for package 'pedantic' declares unused dependencies - please remove them from the project config:
   - console

diff --git a/test-fixtures/pedantic/check-unused-dependency.txt b/test-fixtures/pedantic/check-unused-dependency.txt
@@ -1,18 +1,18 @@
 Reading Spago workspace configuration...
 
-✅ Selecting package to build: pedantic
+✓ Selecting package to build: pedantic
 
 Downloading dependencies...
 Building...
            Src   Lib   All
 Warnings     0     0     0
 Errors       0     0     0
 
-✅ Build succeeded.
+✓ Build succeeded.
 
 Looking for unused and undeclared transitive dependencies...
 
-❌ Found unused and/or undeclared transitive dependencies:
+✘ Found unused and/or undeclared transitive dependencies:
 
 Sources for package 'pedantic' declares unused dependencies - please remove them from the project config:
   - console