Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not use Linter error any more #3703

Open
fabricereix opened this issue Feb 6, 2025 · 2 comments
Open

Do not use Linter error any more #3703

fabricereix opened this issue Feb 6, 2025 · 2 comments
Labels
enhancement New feature or request

Comments

@fabricereix
Copy link
Collaborator

LinterError are not easily maintained and not really useful.
We will simply delete them and simplify the hurlfmt --check usage as follow

$ hurlfmt --check  test.hurl
Would reformat: test.hurl
1 file would be reformatted

$ echo $?
1
@fabricereix fabricereix added the enhancement New feature or request label Feb 6, 2025
@fabricereix
Copy link
Collaborator Author

Multiple input files are supported as follow:

$ hurlfmt --check  test1.hurl test2.hurl test3.hurl
Would reformat: test1.hurl
Would reformat: test2.hurl
2 files would be reformatted

@fabricereix
Copy link
Collaborator Author

The check output (Displaying files to be reformatted) are written to stdout,
while Parsing/IO Error are written to stderr.
It is possible to have both of them (one file in error does not prevent the other files to be processed)

$ hurlfmt --check test1.hurl test2.hurl test3.hurl
error: Parsing space
  --> test1.hurl:1:4
   |
 1 | GET
   |    ^ expecting a space
   |
Would reformat: test2.hurl
1 file would be reformatted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant