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

Uninstall Dockerfilelint #2195

Merged
merged 5 commits into from
Dec 8, 2021
Merged

Uninstall Dockerfilelint #2195

merged 5 commits into from
Dec 8, 2021

Conversation

ferrarimarco
Copy link
Collaborator

@ferrarimarco ferrarimarco commented Nov 29, 2021

Fixes #2179

Proposed Changes

Uninstall Dockerfilelint.

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request

Reviewing Maintainer

  • Label as breaking if this is a large fundamental change
  • Label as either automation, bug, documentation, enhancement, infrastructure, or performance

Requisites

@ferrarimarco ferrarimarco added enhancement New feature or request dependencies Pull requests that update a dependency file labels Nov 29, 2021
@ferrarimarco
Copy link
Collaborator Author

@admiralAwkbar can you kindly update package.json and package-lock.json? I don't have the proper tooling handy on my dev workstation. Thanks!

@zkoppert
Copy link
Contributor

Do you think this should be labelled as a breaking change? I could understand going either way.

@ferrarimarco ferrarimarco added the breaking big changes, potentially causes failure label Nov 30, 2021
@ferrarimarco
Copy link
Collaborator Author

@zkoppert done

@admiralAwkbar
Copy link
Collaborator

@ferrarimarco ask and you shall receive :)

@ferrarimarco
Copy link
Collaborator Author

thanks!

@ferrarimarco
Copy link
Collaborator Author

The PR is ready for review then :)

@admiralAwkbar
Copy link
Collaborator

@ferrarimarco lol somehow removing that broke r....

@ferrarimarco
Copy link
Collaborator Author

I checked, and I think this PR is not the root cause of that. Actually, I think the current main is affected by this issue as well.

@ferrarimarco
Copy link
Collaborator Author

ferrarimarco commented Dec 2, 2021

The root cause is likely that the icu-libs package (upon which the R package depends, doesn't include the libicui18n.so.67 file.

I'll create a separate issue.

@ferrarimarco ferrarimarco self-assigned this Dec 7, 2021
@admiralAwkbar admiralAwkbar merged commit ee99da2 into main Dec 8, 2021
@admiralAwkbar admiralAwkbar deleted the remove-dockerfilelint branch December 8, 2021 15:53
@waja
Copy link

waja commented Dec 21, 2021

This change breaks existing setups where .github/linters/.dockerfilelintrc exists:

2021-12-21 05:34:08 [INFO]   ----------------------------------------------
2021-12-21 05:34:08 [INFO]   ----------------------------------------------
2021-12-21 05:34:08 [INFO]   Linting [DOCKERFILE_HADOLINT] files...
2021-12-21 05:34:08 [INFO]   ----------------------------------------------
2021-12-21 05:34:08 [INFO]   ----------------------------------------------
2021-12-21 05:34:08 [INFO]   ---------------------------
2021-12-21 05:34:08 [INFO]   File:[/github/workspace/.github/linters/.dockerfilelintrc]
2021-12-21 05:34:08 [ERROR]   Found errors in [hadolint] linter!
2021-12-21 05:34:08 [ERROR]   Error code: 1. Command output:
------
/github/workspace/.github/linters/.dockerfilelintrc:1:1 unexpected 'r' expecting '#', ADD, ARG, CMD, COPY, ENTRYPOINT, ENV, EXPOSE, FROM, HEALTHCHECK, LABEL, MAINTAINER, ONBUILD, RUN, SHELL, STOPSIGNAL, USER, VOLUME, WORKDIR, a pragma, end of input, or whitespaces 
------

sarahc23 pushed a commit to 23andMe/super-linter that referenced this pull request May 6, 2022
* Uninstall Dockerfilelint

* remove dockerfilelint packages

* trying ca-certs

Co-authored-by: Admiral Awkbar <admiralawkbar@github.com>
sarahc23 pushed a commit to 23andMe/super-linter that referenced this pull request May 6, 2022
* Uninstall Dockerfilelint

* remove dockerfilelint packages

* trying ca-certs

Co-authored-by: Admiral Awkbar <admiralawkbar@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking big changes, potentially causes failure dependencies Pull requests that update a dependency file enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove Dockerfilelint
4 participants