Skip to content

Fix generator Gemfile after puma upgrade#6060

Merged
tramuntanal merged 3 commits intodecidim:developfrom
Platoniq:fix/generators-gemfile
May 4, 2020
Merged

Fix generator Gemfile after puma upgrade#6060
tramuntanal merged 3 commits intodecidim:developfrom
Platoniq:fix/generators-gemfile

Conversation

@microstudi
Copy link
Copy Markdown
Contributor

@microstudi microstudi commented May 4, 2020

🎩 What? Why?

The version in puma is out of date after its upgrade to version 4.3. This causes the module generator to fail.
Also, bootsnap is required to start the development_app so I'd include it in the Gemfile.
This affects version 0.21 as well.

📌 Related Issues

  • Related to #?
  • Fixes #?

📋 Subtasks

  • Add CHANGELOG entry

@tramuntanal tramuntanal merged commit cdeec34 into decidim:develop May 4, 2020
@tramuntanal
Copy link
Copy Markdown
Contributor

tramuntanal commented May 5, 2020

Hi @microstudi thanks for the fix!
Would you port it to 0.21-stable?

ace pushed a commit to aspgems/decidim that referenced this pull request May 5, 2020
* develop: (29 commits)
  Update Conversations design with decidim-design UI (decidim#6008)
  Add counter of active users to admin dashboard (decidim#5907)
  Show activity graphs on admin dashboard (decidim#6030)
  Update sassc gem version (decidim#6062)
  Fix generator Gemfile after puma upgrade (decidim#6060)
  New Crowdin translations (decidim#6059)
  Add Slovak as a new language (decidim#6039)
  Remove all tests for i18n PRs (decidim#6061)
  Update move up and down buttons after dragging questions when managing questionnaire (decidim#5947)
  Fix using Decidim as a provider for omniauth authentication (decidim#6042)
  Add redesign for responsive public profile navigation tabs (decidim#6032)
  Add versioning pages to initiatives (decidim#5935)
  Notify users when roles get assigned (decidim#5886)
  Improve the budget page and the project card (decidim#5809)
  New Crowdin translations (decidim#6050)
  New Crowdin translations (decidim#6046)
  Ignore jobs on locales branches (decidim#6047)
  Automatic task for deleting Meeting Inscription data (decidim#5989)
  New Crowdin translations (decidim#5877)
  Ignore builds on Crowdin PRs (decidim#6037)
  ...
microstudi added a commit that referenced this pull request May 6, 2020
* sync puma version in generator Gemfile

* Add changelog

* add bootsnap as required
@microstudi
Copy link
Copy Markdown
Contributor Author

microstudi commented May 6, 2020

backport done @tramuntanal #6072

tramuntanal added a commit that referenced this pull request May 6, 2020
* Fix generator Gemfile after puma upgrade (#6060)

* sync puma version in generator Gemfile

* Add changelog

* add bootsnap as required

* Apply Js bundle sanity by running webpack task

Co-authored-by: Oliver Valls <oliver.vh@coditramuntana.com>
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.

3 participants