Skip to content

Add periods to PR template list entries that didn't have it #628

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

Merged
merged 2 commits into from
Feb 3, 2025

Conversation

mokagio
Copy link
Contributor

@mokagio mokagio commented Feb 2, 2025

This way, all entries have a period and the template is homogeneous.

Just something that caught my eye in #627. Not sure why it too so long to notice...

Below is the template as generate with the version on trunk 👇

Checklist before requesting a review

  • Run bundle exec rubocop to test for code style violations and recommendations
  • Add Unit Tests (aka specs/*_spec.rb) if applicable
  • Run bundle exec rspec to run the whole test suite and ensure all your tests pass
  • Make sure you added an entry in the CHANGELOG.md file to describe your changes under the appropriate existing ### subsection of the existing ## Trunk section.
  • If applicable, add an entry in the MIGRATION.md file to describe how the changes will affect the migration from the previous major version and what the clients will need to change and consider.

This way, all entries have a period and the template is homogeneous.
@mokagio mokagio enabled auto-merge February 2, 2025 23:19
@mokagio mokagio requested a review from a team February 2, 2025 23:20
Copy link
Contributor

@AliSoftware AliSoftware left a comment

Choose a reason for hiding this comment

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

Easy one 🙂

@dangermattic
Copy link
Collaborator

1 Warning
⚠️ Please add an entry in the CHANGELOG.md file to describe the changes made by this PR

Generated by 🚫 Danger

@mokagio mokagio merged commit bcc522b into trunk Feb 3, 2025
6 checks passed
@mokagio mokagio deleted the mokagio/template-fix branch February 3, 2025 22:27
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