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

Fix Bootsnap configuration #11483

Merged
merged 1 commit into from
Sep 6, 2023
Merged

Fix Bootsnap configuration #11483

merged 1 commit into from
Sep 6, 2023

Conversation

alecslupu
Copy link
Contributor

🎩 What? Why?

On a closer look on the initialization process, i have noticed that our bootsnap config is not working properly.

As per rails bootsnap.rb template, we can see that we have

require "bootsnap/setup" # Speed up boot time by caching expensive operations.

Whereas, in our generator we have

gsub_file "config/boot.rb", %r{require 'bootsnap/setup'.*$}, <<~RUBY.rstrip

As we can see rails official version is using double quotes whereas we are just using single quotes.

Testing

  1. On a development application open "config/bootsnap.rb"
  2. See the decidim related config is not present
  3. Apply the patch
  4. Regenerate application
  5. Open the new "config/bootsnap.rb"
  6. See decidim related config there.
  7. Repeat 1 to 6 for the test application.

♥️ Thank you!

@alecslupu alecslupu added module: generators type: fix PRs that implement a fix for a bug labels Aug 16, 2023
@alecslupu alecslupu marked this pull request as ready for review August 16, 2023 16:31
@alecslupu alecslupu requested a review from a team August 17, 2023 16:31
@alecslupu alecslupu added this to the 0.28.0 milestone Sep 5, 2023
Copy link
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

👍🏽 I tried it locally and it works as expected, great finding!

@andreslucena andreslucena merged commit 035438f into develop Sep 6, 2023
34 checks passed
@andreslucena andreslucena deleted the chore/fix-bootsnap branch September 6, 2023 07:46
entantoencuanto added a commit that referenced this pull request Sep 6, 2023
* develop:
  Remove duplication from amendments events specs (#11553)
  Remove duplication from elections' user roles forms (#11548)
  Update Node.js from v16.13.0 to v18.17.1 (#11564)
  Remove duplication from stats presenters (#11551)
  Fix Bootsnap configuration (#11483)
entantoencuanto added a commit that referenced this pull request Sep 8, 2023
…gn-staging

* fix/activities-block-follow-button: (27 commits)
  Add tests to follow button in processes and assemblies landing page
  Add follow button to participatory spaces last activities content block
  Remove duplication from participatory spaces publications controllers (#11549)
  Fix the a11y tool icons with redesign (#11175)
  Remove duplication from amendments events specs (#11553)
  Remove duplication from elections' user roles forms (#11548)
  Update Node.js from v16.13.0 to v18.17.1 (#11564)
  Remove duplication from stats presenters (#11551)
  Fix Bootsnap configuration (#11483)
  Remove duplication for add questions specs examples (#11559)
  Remove duplication from invites queries (#11552)
  Fix typos and copy-paste errors from comments and examples (#11536)
  Fix conference venues meetings visibility (#11542)
  Add recognition to BrowserStack in the README (#11546)
  Remove unused view hook for `:upcoming_meeting_for_card` (#11543)
  Remove unused dependency: `wicked` (#11150)
  Clean-up initiatives signature URLs and methods (#11545)
  Refactor initiative signing wizard (#10731)
  Fix Permissions screen on budgets throw errors (#11532)
  Redesign: read more literal (#11516)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: generators type: fix PRs that implement a fix for a bug
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants