de-emoji

IanButterworth · IanButterworth · commit 719fd82f6fe3 · 2025-08-28T21:35:57.000-04:00
diff --git a/MIGRATION.md b/MIGRATION.md
@@ -6,15 +6,15 @@ This guide helps you migrate from the deprecated direct upload functionality to
 
 Coverage.jl has been modernized to work with the official uploaders from Codecov and Coveralls, as both services have deprecated support for 3rd party uploaders.
 
-### Before (Deprecated ❌)
+### Before (Deprecated)
 ```julia
 using Coverage
 fcs = process_folder("src")
-Codecov.submit(fcs)           # ❌ Deprecated
-Coveralls.submit(fcs)         # ❌ Deprecated
+Codecov.submit(fcs)           # Deprecated
+Coveralls.submit(fcs)         # Deprecated
 ```
 
-### After (Modern ✅)
+### After (Modern)
 ```julia
 using Coverage
 fcs = process_folder("src")
diff --git a/README.md b/README.md
@@ -9,10 +9,10 @@ Coverage.jl
 
 **Coverage.jl has been modernized** to work with the official uploaders from Codecov and Coveralls.
 The package now provides:
-- 🔄 **Coverage data processing** using CoverageTools.jl
-- 📤 **Export functionality** for official uploaders
-- 🚀 **Automated upload helpers** for CI environments
-- 📋 **Helper scripts** for easy integration
+- **Coverage data processing** using CoverageTools.jl
+- **Export functionality** for official uploaders
+- **Automated upload helpers** for CI environments
+- **Helper scripts** for easy integration
 
 > [!NOTE]
 > **Coverage.jl now uses official uploaders from Codecov and Coveralls** for better reliability and future compatibility. The familiar `Codecov.submit()` and `Coveralls.submit()` functions continue to work seamlessly.
diff --git a/scripts/script_utils.jl b/scripts/script_utils.jl
@@ -62,7 +62,7 @@ function main_with_error_handling(main_func)
     try
         return main_func()
     catch e
-        println("❌ Error: $(sprint(Base.display_error, e))")
+        println("Error: $(sprint(Base.display_error, e))")
         return 1
     end
 end
diff --git a/scripts/upload_codecov.jl b/scripts/upload_codecov.jl
@@ -44,7 +44,7 @@ function main()
     fcs = process_folder(folder)
 
     if isempty(fcs)
-        println("❌ No coverage data found in folder: $folder")
+        println("No coverage data found in folder: $folder")
         return 1
     end
 
diff --git a/scripts/upload_coverage.jl b/scripts/upload_coverage.jl
@@ -55,10 +55,10 @@ function main()
     # Check results
     if service == :both
         success = all(values(result))
-        println(success ? "✅ All uploads successful" : "❌ Some uploads failed")
+        println(success ? "All uploads successful" : "Some uploads failed")
     else
         success = result
-        println(success ? "✅ Upload successful" : "❌ Upload failed")
+        println(success ? "Upload successful" : "Upload failed")
     end
 
     return success ? 0 : 1
diff --git a/scripts/upload_coveralls.jl b/scripts/upload_coveralls.jl
@@ -88,23 +88,23 @@ function main()
         args = parse_commandline()
 
         # Show configuration
-        println("📊 Coveralls Upload Configuration")
+        println("Coveralls Upload Configuration")
         println("Folder: $(args["folder"])")
         println("Format: $(args["format"])")
         println("Token: $(args["token"] !== nothing ? "<provided>" : "from environment")")
         println("Dry run: $(args["dry-run"])")
         println()
 
         # Process coverage
-        println("🔄 Processing coverage data...")
+        println("Processing coverage data...")
         fcs = process_folder(args["folder"])
 
         if isempty(fcs)
-            println("❌ No coverage data found in folder: $(args["folder"])")
+            println("No coverage data found in folder: $(args["folder"])")
             exit(1)
         end
 
-        println("✅ Found coverage data for $(length(fcs)) files")
+        println("Found coverage data for $(length(fcs)) files")
 
         # Upload to Coveralls
         success = upload_to_coveralls(fcs;
@@ -114,15 +114,15 @@ function main()
         )
 
         if success
-            println("🎉 Successfully uploaded to Coveralls!")
+            println("Successfully uploaded to Coveralls!")
             exit(0)
         else
-            println("❌ Failed to upload to Coveralls")
+            println("Failed to upload to Coveralls")
             exit(1)
         end
 
     catch e
-        println("❌ Error: $(sprint(Base.display_error, e))")
+        println("Error: $(sprint(Base.display_error, e))")
         exit(1)
     end
 end
diff --git a/src/ci_integration_functions.jl b/src/ci_integration_functions.jl
@@ -352,14 +352,14 @@ function finish_coveralls_parallel(; token=nothing, build_num=nothing)
         )
 
         if response.status == 200
-            @info "✅ Successfully signaled parallel job completion to Coveralls"
+            @info "Successfully signaled parallel job completion to Coveralls"
             return true
         else
-            @error "❌ Failed to signal parallel completion" status=response.status
+            @error "Failed to signal parallel completion" status=response.status
             return false
         end
     catch e
-        @error "❌ Error signaling parallel completion to Coveralls" exception=e
+        @error "Error signaling parallel completion to Coveralls" exception=e
         return false
     end
 end
diff --git a/test/runtests.jl b/test/runtests.jl
@@ -779,13 +779,13 @@ withenv(
 
                         # The fact that it started without immediate crash is good enough
                         @test true  # If we get here, the executable at least started
-                        @info "✅ Codecov uploader executable verified (can start)"
+                        @info "Codecov uploader executable verified (can start)"
                     catch e
                         # If it fails with a specific error message, that's actually good
                         # (means it's running but needs proper args/config)
                         if isa(e, ProcessFailedException) && e.procs[1].exitcode != 127
                             @test true  # Non-127 exit means executable works (127 = not found)
-                            @info "✅ Codecov uploader executable verified (exits with expected error)"
+                            @info "Codecov uploader executable verified (exits with expected error)"
                         else
                             @warn "Codecov uploader may not be functional" exception=e
                             # Don't fail the test - platform issues might prevent execution
@@ -797,7 +797,7 @@ withenv(
                     try
                         output = read(`$exe_path --version`, String)
                         @test !isempty(strip(output))
-                        @info "✅ Codecov uploader version: $(strip(output))"
+                        @info "Codecov uploader version: $(strip(output))"
                     catch e
                         # Version command might not be available, that's ok
                         @debug "Version command not available" exception=e
@@ -842,12 +842,12 @@ withenv(
 
                         # The fact that it started without immediate crash is good enough
                         @test true
-                        @info "✅ Coveralls reporter executable verified (can start)"
+                        @info "Coveralls reporter executable verified (can start)"
                     catch e
                         # If it fails with a specific error message, that's actually good
                         if isa(e, ProcessFailedException) && e.procs[1].exitcode != 127
                             @test true  # Non-127 exit means executable works
-                            @info "✅ Coveralls reporter executable verified (exits with expected error)"
+                            @info "Coveralls reporter executable verified (exits with expected error)"
                         else
                             @warn "Coveralls reporter may not be functional" exception=e
                             @test_skip "Coveralls executable functionality"
@@ -858,13 +858,13 @@ withenv(
                     try
                         output = read(`$exe_path --version`, String)
                         @test !isempty(strip(output))
-                        @info "✅ Coveralls reporter version: $(strip(output))"
+                        @info "Coveralls reporter version: $(strip(output))"
                     catch e
                         # Try alternative version command
                         try
                             output = read(`$exe_path version`, String)
                             @test !isempty(strip(output))
-                            @info "✅ Coveralls reporter version: $(strip(output))"
+                            @info "Coveralls reporter version: $(strip(output))"
                         catch e2
                             @debug "Version command not available" exception=e2
                         end
@@ -908,13 +908,13 @@ withenv(
                                 end
 
                                 @test true
-                                @info "✅ Codecov can process LCOV files"
+                                @info "Codecov can process LCOV files"
                             catch e
                                 if isa(e, ProcessFailedException)
                                     # Check if it's a validation error vs system error
                                     if e.procs[1].exitcode != 127  # Not "command not found"
                                         @test true  # File was processed, error might be network/auth related
-                                        @info "✅ Codecov processed file (expected error without token)"
+                                        @info "Codecov processed file (expected error without token)"
                                     else
                                         @test_skip "Codecov executable system error"
                                     end
@@ -950,7 +950,7 @@ withenv(
                                 end
 
                                 @test true
-                                @info "✅ Coveralls can process LCOV files"
+                                @info "Coveralls can process LCOV files"
                             catch e
                                 # Try without --dry-run flag (might not be supported)
                                 try
@@ -961,7 +961,7 @@ withenv(
                                         kill(result)
                                     end
                                     @test true
-                                    @info "✅ Coveralls executable responds to commands"
+                                    @info "Coveralls executable responds to commands"
                                 catch e2
                                     @test_skip "Coveralls file processing test failed"
                                 end
