Skip to content

chore(giga): remove mentions of experimental and evmone#3411

Open
arajasek wants to merge 1 commit intomainfrom
asr/remove-comment
Open

chore(giga): remove mentions of experimental and evmone#3411
arajasek wants to merge 1 commit intomainfrom
asr/remove-comment

Conversation

@arajasek
Copy link
Copy Markdown
Contributor

@arajasek arajasek commented May 8, 2026

Describe your changes and provide context

The Giga executor is no longer experimental, so we should remove mentions of that in comments and config.

It's also not (strictly) evmone-based, so we remove mentions of that as well.

Testing performed to validate your change


Note

Low Risk
Comment/documentation-only updates around the Giga executor; no runtime behavior or configuration keys change, so risk is low.

Overview
Cleans up Giga executor messaging by removing references to it being experimental or strictly evmone-based across app comments, config templates, docs, scripts, and integration test descriptions.

No functional changes: the GIGA_EXECUTOR/GIGA_OCC flags, TOML keys, and transaction execution routing remain the same; only user-facing wording and log/script text is updated.

Reviewed by Cursor Bugbot for commit 22e2b4c. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMay 8, 2026, 7:26 PM

@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.24%. Comparing base (0543e0e) to head (22e2b4c).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3411   +/-   ##
=======================================
  Coverage   59.24%   59.24%           
=======================================
  Files        2110     2110           
  Lines      174149   174149           
=======================================
  Hits       103175   103175           
  Misses      62041    62041           
  Partials     8933     8933           
Flag Coverage Δ
sei-chain-pr 50.09% <ø> (?)
sei-db 70.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
app/app.go 69.68% <ø> (ø)
giga/executor/config/config.go 60.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@arajasek arajasek force-pushed the asr/remove-comment branch from 667565b to 22e2b4c Compare May 8, 2026 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant