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

Create executable bug report Rails application #593

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

seanpdoyle
Copy link
Contributor

This template draws inspiration from Rails' bug report templates.

@seanpdoyle
Copy link
Contributor Author

@jorgemanrubia it'd be beneficial to pair this script with a .github/ISSUE_TEMPLATE.md file like Rails uses. There is a new version of GitHub issue templates that could be worth investigating.

@seanpdoyle seanpdoyle force-pushed the bug-report-template branch 7 times, most recently from 430f58b to 3baf70f Compare February 29, 2024 12:01
@@ -29,6 +29,9 @@ jobs:
rubygems: latest
bundler-cache: true

- name: Run Bug Template Tests
run: ruby bug_report_template.rb || ruby bug_report_template.rb
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The first execution fails with:

/opt/hostedtoolcache/Ruby/3.0.6/x64/lib/ruby/site_ruby/3.0.0/bundler/runtime.rb:304:in `check_for_activated_spec!': You have already activated stringio 3.0.1, but your Gemfile requires stringio 3.1.0. Since stringio is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports stringio as a default gem. (Gem::LoadError)

I'm not sure why, but executing a second time resolves that problem.

@seanpdoyle seanpdoyle force-pushed the bug-report-template branch from 7e61ae7 to 1d40e37 Compare July 13, 2024 09:57
@seanpdoyle
Copy link
Contributor Author

@jorgemanrubia @brunoprietog are either of you available to review this proposal? I think it'd improve this repository's issue-opening process.

Copy link
Member

@jorgemanrubia jorgemanrubia left a comment

Choose a reason for hiding this comment

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

Thanks @seanpdoyle

@jorgemanrubia jorgemanrubia merged commit 9b17a3b into hotwired:main Jul 19, 2024
15 checks passed
@seanpdoyle seanpdoyle deleted the bug-report-template branch July 19, 2024 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants