Skip to content

Conversation

drinkcat
Copy link
Collaborator

Arguably this should be part of the CI, but I'm not sure how to run it outside of precommit.

Arguably this should be part of the CI, but I'm not sure how
to run it outside of precommit.
@sylvestre
Copy link
Contributor

i have a patch for flt check in github, easy :)

Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/timeout/timeout (passes in this run but fails in the 'main' branch)

@sylvestre
Copy link
Contributor

Are you sure they are really issues?

@drinkcat
Copy link
Collaborator Author

I... don't know, I happened to touch another ftl file and this popped up:

$ git checkout upstream/main 
$ sed -i -e 's/about/above/' src/uu/stat/locales/en-US.ftl
$ git commit -a
...
fluent_linter............................................................Failed
- hook id: fluent_linter
- exit code: 1

Files to analyze: 199.

            File path: src/uu/stat/locales/en-US.ftl
            Message ID: -
            Position: line 56 column 1
            Error (GC02): Group comments (##) should be followed by one empty line.

            File path: src/uu/stat/locales/en-US.ftl
            Message ID: -
            Position: line 66 column 1
            Error (GC02): Group comments (##) should be followed by one empty line.
...

@sylvestre sylvestre merged commit fb4d611 into uutils:main Jul 28, 2025
78 checks passed
@drinkcat drinkcat deleted the stat-fluent branch July 28, 2025 12:12
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.

2 participants