-
Notifications
You must be signed in to change notification settings - Fork 0
Release/1.10.0 #75
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
Release/1.10.0 #75
Conversation
Sync Release 1.9.0
Sync with hotfix 1.9.1
* Hotfix 1.9.1 (#66) * Fix RST syntax error * Remove orphaned print statement * Update CL * Bump version 1.9.1 -> 1.9.2 Co-authored-by: Steffengreiner <[email protected]>
The summary file is now searched within the summary folder
Sync with hotfix 1.9.3
Steffengreiner
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heya @wow-such-code, thanks for preparing the release! 👍
I found some minor issues that should be addressed before it's ready for merging.
- Instead of bumping the version by hand, i'd suggest using the qube bump-version functionality to keep the version in the qube files(such as qube.yml and qube.cfg) also in sync.
- This is a Nit but i'd appreciate it if the changelog date matches the release date
Co-authored-by: Steffengreiner <[email protected]>
Steffengreiner
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice Qube Bump-Version, however it duplicated an entry in the changelog. 🙈
CHANGELOG.rst
Outdated
| 1.10.0 (2021-09-13) | ||
| ------------------- | ||
|
|
||
| **Added** | ||
|
|
||
| **Fixed** | ||
|
|
||
| **Dependencies** | ||
|
|
||
| **Deprecated** | ||
|
|
||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess qube duplicated the Changelog entry? 😕
| 1.10.0 (2021-09-13) | |
| ------------------- | |
| **Added** | |
| **Fixed** | |
| **Dependencies** | |
| **Deprecated** |
Steffengreiner
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM ![]()
No description provided.