Skip to content

lint: Add gci formatter#6536

Merged
gmalouf merged 2 commits intoalgorand:masterfrom
cce:use-gci-again
Feb 4, 2026
Merged

lint: Add gci formatter#6536
gmalouf merged 2 commits intoalgorand:masterfrom
cce:use-gci-again

Conversation

@cce
Copy link
Copy Markdown
Contributor

@cce cce commented Jan 29, 2026

Summary

Update for deterministic import order formatting matching style used by go-algorand-sdk and indexer. (Continuation of #3734)

Test Plan

No code changed, just import ordering.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.70%. Comparing base (172f19d) to head (36c57fb).
⚠️ Report is 4 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6536      +/-   ##
==========================================
- Coverage   47.90%   47.70%   -0.20%     
==========================================
  Files         662      655       -7     
  Lines       87991    87906      -85     
==========================================
- Hits        42149    41938     -211     
- Misses      43085    43196     +111     
- Partials     2757     2772      +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cce cce requested a review from algorandskiy February 3, 2026 18:18
Copy link
Copy Markdown
Contributor

@algorandskiy algorandskiy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but lets merge after #6400

@cce cce requested a review from gmalouf February 3, 2026 21:11
@gmalouf gmalouf merged commit a64eadd into algorand:master Feb 4, 2026
40 checks passed
@cce cce deleted the use-gci-again branch February 9, 2026 16:03
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.

3 participants