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

Update puma to 4.3.6 #1533

Merged
merged 2 commits into from Nov 19, 2020
Merged

Update puma to 4.3.6 #1533

merged 2 commits into from Nov 19, 2020

Conversation

ogirginc
Copy link
Contributor

Because

There was a bug on Puma version 4.3.5, causing it to fail when trying to
compile. It was fixed with #2304.

This addresses

This commit updates Gemfile.lock and removes the now redundant
explanations on readme.

There was a bug on Puma version 4.3.5, causing it to fail when trying to
compile. It was fixed with [#2304].

This commit updates Gemfile.lock and removes the now redundant
explanations on readme.

[#2304]: puma/puma#2304
@kitallis
Copy link
Contributor

Hi @ogirginc, thanks a ton for the submission! We'll try and find sometime this week to check if nothing breaks because of the puma upgrade, but overall, seems like it's going to cut down a bunch of noise from the setup procedures.

Heads-up, the team is on a break in the last week of November, so if we don't get a chance to try this, we might only get to it by the first week of December.

Cheers!

@ogirginc
Copy link
Contributor Author

Hi @kitallis. Thanks for the heads up and no worries at all!

Technically, it should not break anything according to the changelog, which has only 2 bug fixes:

  • Explicitly include ctype.h to fix compilation warning and build error on macOS with Xcode 12 (#2304)
  • Don't require json at boot (#2269)

Have a nice & safe break!

@rsanheim
Copy link
Contributor

I just hit the puma issue again while bootstrapping a new M1 mac, so I'm going to take a look at merging this...

Copy link
Contributor

@rsanheim rsanheim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good @ogirginc ! Thanks for the contribution.

@rsanheim rsanheim merged commit 46a8a06 into simpledotorg:master Nov 19, 2020
@rsanheim
Copy link
Contributor

side note: I had to temporarily disable some statue check requirements, as I don't believe our workflows run Standardrb checks or the "CI push" check from forked contributions.

@ogirginc
Copy link
Contributor Author

the "CI push" check from forked contributions.

When I pushed #1532, it did actually worked. However, Github showed me "This branch is out-of-date with the base branch" warning and I pressed the "Update Branch" button which created a merge commit. To prevent merge commit, I have pulled master, rebased and pushed but this time CI did not run. 🤷 Just thought, I should share as it might help somehow.

@ogirginc ogirginc deleted the puma-4.3.6 branch November 30, 2020 17:16
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

3 participants