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

Bullet Train Update: 1.7.23 #40

Merged
merged 39 commits into from
Oct 18, 2024
Merged

Bullet Train Update: 1.7.23 #40

merged 39 commits into from
Oct 18, 2024

Conversation

jagthedrummer
Copy link
Contributor

Update of the Bullet Train Starter Repo to version 1.7.23

Please note: You should thorougly review this PR.

🎉 There are no conflicts detected in this PR!

Auto-generated by .github/workflows/update-bullet-train.yml

depfu bot and others added 30 commits August 22, 2024 06:18
Fixes bullet-train-co/bullet_train#1663

We shouldn't replace the whole Rails' app's RouteSet, particularly when the `@routes`
instance variable already provides the hook.

Instead we can mirror what Rails does internally:
https://github.com/rails/rails/blob/a59ddc714fa0403cbd8c7ae6abdec2f71ee7b63f/actionpack/test/controller/test_case_test.rb#L1224C1-L1236C8
Co-authored-by: jagthedrummer <jagthedrummer@users.noreply.github.com>
depfu bot and others added 9 commits September 18, 2024 06:19
…705)

* select2_select_selenium testing: allow a field label to be wrapped in a div

* find_stimulus_controller_for_label: allow label to have a wrapper div

* invitation_lists_test: avoid sibling to find input

* find_stimulus_controller_for_label: it hurts to do this, but go one parent up

* find_stimulus_controller_for_label: use ancestor and simplify if not expecting wrapper match

* fix standardrb

* find_stimulus_controller_for_label: revert to old xpath for going up two parents
* Update gitpod.Dockerfile

 - Move to Ruby 3.3.4
 - add graphviz as a package to install as it is missing on db migration

* Update .gitpod.yml

Make rails init code work seamlessly with gitpod
Co-authored-by: jagthedrummer <jagthedrummer@users.noreply.github.com>
# Conflicts:
#	Gemfile.lock
@jagthedrummer jagthedrummer merged commit 02867d5 into main Oct 18, 2024
9 checks passed
@jagthedrummer jagthedrummer deleted the bullet-train-update/1.7.23 branch October 18, 2024 19:29
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.

5 participants