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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade standard to 1.34 #1346

Closed
wants to merge 3 commits into from
Closed

Upgrade standard to 1.34 #1346

wants to merge 3 commits into from

Conversation

jim
Copy link
Member

@jim jim commented Feb 18, 2024

What it does

Upgrades standard and fixes a few issues from a new default setting. It promotes the use of anonymous block arguments forwarding, which... 馃し馃徎

Standard only found some of the uses of &block in the codebase, which isn't great. It looks like there are some issues with the changes in Ruby 3.3 that may be trickling down here based on Naming/BlockForwarding with anonymous block parameter (Ruby 3.3.0)

We might want to wait on this one to let some of these issues get shaken out.

dependabot bot and others added 2 commits February 18, 2024 16:29
Bumps [standard](https://github.com/standardrb/standard) from 1.33.0 to 1.34.0.
- [Release notes](https://github.com/standardrb/standard/releases)
- [Changelog](https://github.com/standardrb/standard/blob/main/CHANGELOG.md)
- [Commits](standardrb/standard@v1.33.0...v1.34.0)

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

Signed-off-by: dependabot[bot] <support@github.com>
@jim jim changed the title Jim standard 1 34 Upgrade standard to 1.34 Feb 18, 2024
@jim jim requested a review from phinze February 18, 2024 16:58
@jim jim marked this pull request as draft February 18, 2024 17:36
@phinze
Copy link
Contributor

phinze commented Feb 20, 2024

+1 to hold off on this one - i think we can tell dependabot to skip a version

@phinze
Copy link
Contributor

phinze commented Feb 20, 2024

Ignored in #1341 and we can let the next update swing around 馃憤馃徎

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.

None yet

2 participants