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

Bump haml_lint from 0.40.1 to 0.41.0 #932

Merged
merged 5 commits into from
Sep 26, 2022

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 26, 2022

Bumps haml_lint from 0.40.1 to 0.41.0.

Release notes

Sourced from haml_lint's releases.

HAML-Lint 0.41.0

  • Add support for HAML 6 beta
  • Fix Ruby extractor to keep newlines in certain cases
Changelog

Sourced from haml_lint's changelog.

0.41.0

  • Add support for HAML 6 beta
  • Fix Ruby extractor to keep newlines in certain cases
Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [haml_lint](https://github.com/sds/haml-lint) from 0.40.1 to 0.41.0.
- [Release notes](https://github.com/sds/haml-lint/releases)
- [Changelog](https://github.com/sds/haml-lint/blob/main/CHANGELOG.md)
- [Commits](sds/haml-lint@v0.40.1...v0.41.0)

---
updated-dependencies:
- dependency-name: haml_lint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file ruby Pull requests that update Ruby code labels Sep 26, 2022
@deivid-rodriguez deivid-rodriguez temporarily deployed to bundler-site-dependabot-njekl5 September 26, 2022 08:01 Inactive
@deivid-rodriguez deivid-rodriguez temporarily deployed to bundler-site-dependabot-njekl5 September 26, 2022 08:17 Inactive
@olleolleolle
Copy link
Member

olleolleolle commented Sep 26, 2022

(I searched around for the problem, and I thank the existing reported Issues for being so clear! rubocop/rubocop#9864 )

HAML-Lint seems not to support all kinds of arguments. Perhaps we must make this uglier to make it pass?

Update: sds/haml-lint#333 the workaround is to fully ignore Layout/ArgumentAlignment in our RuboCop configuration, it seems.

I will make an attempted disable, and see if we can get the linting to pass like that. (Yes, that passed.)

Since we had 1 and only 1 such linting failure, perhaps we could rephrase the HAML code locally, perhaps using local variables - that way we could re-enable the RuboCop rule and enjoy its benefits elsewhere.

This is a workaround, to make haml-lint linting pass.
@deivid-rodriguez deivid-rodriguez temporarily deployed to bundler-site-dependabot-njekl5 September 26, 2022 08:28 Inactive
@deivid-rodriguez deivid-rodriguez temporarily deployed to bundler-site-dependabot-njekl5 September 26, 2022 14:40 Inactive
@olleolleolle
Copy link
Member

Locally, I was able to make it pass, by making it slightly uglier, to get it to pass.

➜  bundler-site git:(dependabot/bundler/haml_lint-0.41.0) bundle exec haml-lint

93 files inspected, 0 lints detected

@olleolleolle olleolleolle merged commit 7d1bc98 into master Sep 26, 2022
@olleolleolle olleolleolle deleted the dependabot/bundler/haml_lint-0.41.0 branch September 26, 2022 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file ruby Pull requests that update Ruby code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants