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

Fails on latest ubuntu-20 image #54

Closed
tesharp opened this issue Jun 16, 2021 · 5 comments · Fixed by #55
Closed

Fails on latest ubuntu-20 image #54

tesharp opened this issue Jun 16, 2021 · 5 comments · Fixed by #55

Comments

@tesharp
Copy link

tesharp commented Jun 16, 2021

After update to version 20210614.1 of ubuntu 20.04 image, which has git 2.34.0 installed it fails with following error message:

fatal: ls-files -i must be used with either -o or -c
parse error: Invalid numeric literal at line 1, column 7
reviewdog: parse error: failed to unmarshal rdjson (DiagnosticResult): proto: syntax error (line 1:1): unexpected token

Not running successfully on ubuntu-latest now

@tsingh-wavefin
Copy link

Also having this issue. It's difficult to work around, as we can't pin old image versions in GitHub virtual environments. Also, this affects both the Ubuntu and MacOS images.

@odannyc
Copy link

odannyc commented Jun 17, 2021

Also running into this issue on latest version. Error:

  fatal: ls-files -i must be used with either -o or -c
  parse error: Invalid numeric literal at line 1, column 7
  reviewdog: failed to unmarshal rdjson (DiagnosticResult): proto: syntax error (line 1:1): unexpected token 

@mgrachev mgrachev linked a pull request Jun 18, 2021 that will close this issue
@mgrachev
Copy link
Member

mgrachev commented Jun 18, 2021

Hi there!

Can anyone test the new version of action-hadolint v1.20.1? Does it work?

@tsingh-wavefin
Copy link

@mgrachev Yep, can confirm that this version is working now. Thanks for letting us know!

@mgrachev
Copy link
Member

Thank you for your feedback 👍

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 a pull request may close this issue.

4 participants