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

Introduce Changelog and PR template #765

Merged
merged 8 commits into from Jun 11, 2020
Merged

Conversation

jaredbeck
Copy link
Contributor

@jaredbeck jaredbeck commented Jun 10, 2020

Hi Michael. I just upgraded to 3, and it seems to be working fine. I thought I'd add a changelog to help other people to upgrade.

[Fixes #677]

The addition of a PR template was requested in
grosser#677

Here, I'm copying the one we use at authlogic, as a starting
point. Of course, we can customize it to the needs of parallel_tests.
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@grosser
Copy link
Owner

grosser commented Jun 10, 2020

yah, this should help :)

jaredbeck and others added 4 commits June 10, 2020 19:41
Co-authored-by: Michael Grosser <michael@grosser.it>
Co-authored-by: Michael Grosser <michael@grosser.it>
Co-authored-by: Michael Grosser <michael@grosser.it>
Co-authored-by: Michael Grosser <michael@grosser.it>
CHANGELOG.md Outdated Show resolved Hide resolved
### Dependencies

- Drop ruby 2.3 support (#760)
- Drop ruby 2.2 support (#759)
Copy link
Owner

Choose a reason for hiding this comment

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

maybe just list these under breaking ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Happy to move them if you want. In my own projects, I don't like to set the expectation that dependency updates will always mean major bumps. And, SemVer does not consider dependency updates to be breaking changes (as long as the public API stays the same).

jaredbeck and others added 2 commits June 10, 2020 19:53
Co-authored-by: Michael Grosser <michael@grosser.it>
Co-authored-by: Michael Grosser <michael@grosser.it>
@grosser grosser merged commit f997437 into grosser:master Jun 11, 2020
@grosser
Copy link
Owner

grosser commented Jun 11, 2020

thx!

@jaredbeck jaredbeck deleted the changelog branch June 11, 2020 00:31

### Breaking Changes

- Cucumber 3 support removed (#762)
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi! I believe cucumber 3 should still work?

Copy link
Owner

Choose a reason for hiding this comment

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

ah yes ... this was just the sorting part

Copy link
Contributor

Choose a reason for hiding this comment

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

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.

Add changelog?
3 participants