Skip to content

Conversation

@wow-such-code
Copy link
Member

No description provided.

sven1103 and others added 8 commits July 20, 2021 08:44
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
@wow-such-code
Copy link
Member Author

Copy link
Contributor

@Steffengreiner Steffengreiner left a 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.

  1. 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.
  2. This is a Nit but i'd appreciate it if the changelog date matches the release date

Copy link
Contributor

@Steffengreiner Steffengreiner left a 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
Comment on lines 62 to 73
1.10.0 (2021-09-13)
-------------------

**Added**

**Fixed**

**Dependencies**

**Deprecated**


Copy link
Contributor

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? 😕

Suggested change
1.10.0 (2021-09-13)
-------------------
**Added**
**Fixed**
**Dependencies**
**Deprecated**

Copy link
Contributor

@Steffengreiner Steffengreiner left a comment

Choose a reason for hiding this comment

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

LGTM :godmode:

@wow-such-code wow-such-code merged commit d0856b9 into master Sep 13, 2021
@jenniferboedker jenniferboedker deleted the release/1.10.0 branch February 8, 2022 12:44
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