Skip to content

Comments

chore(#3271): add new format-check, rename style-fix to format-fix#3273

Merged
alex-courtis merged 8 commits intomasterfrom
3271
Feb 17, 2026
Merged

chore(#3271): add new format-check, rename style-fix to format-fix#3273
alex-courtis merged 8 commits intomasterfrom
3271

Conversation

@alex-courtis
Copy link
Member

@alex-courtis alex-courtis commented Feb 17, 2026

Fixes #3271
Path of least resistence as noted above.

Summary:

  • Install CodeFormat during CI. Used binary as luarocks compiles it very slowly.
  • rename style-fix to format-fix to avoid ambiguity
  • add style-check in CI:
    • run style-fix
    • fail on git diff
  • fixed any formatting issues

CI failure as expected, before formatting was fixed: https://github.com/nvim-tree/nvim-tree.lua/actions/runs/22083850314/job/63814273749?pr=3273

@alex-courtis alex-courtis requested a review from Uanela February 17, 2026 01:53
@alex-courtis alex-courtis marked this pull request as draft February 17, 2026 01:53
@alex-courtis alex-courtis marked this pull request as ready for review February 17, 2026 02:32
Copy link
Collaborator

@Uanela Uanela left a comment

Choose a reason for hiding this comment

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

LGTM

@alex-courtis alex-courtis merged commit e11ce83 into master Feb 17, 2026
4 checks passed
@alex-courtis alex-courtis deleted the 3271 branch February 17, 2026 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

style-check and style-fix do not agree

3 participants