Skip to content

Fix version mismatch#121

Merged
unkcpz merged 1 commit into
aiidalab:masterfrom
danielhollas:fix-version
Nov 24, 2022
Merged

Fix version mismatch#121
unkcpz merged 1 commit into
aiidalab:masterfrom
danielhollas:fix-version

Conversation

@danielhollas

Copy link
Copy Markdown
Contributor

In the last release, the version was not actually updated.

This didn't broke anything, because this package is not published anywhere, and is installed in aiidalab-docker-stack directly from the git repo based on the git version tag, which was correct. But if you try to identify the version in the current image

mamba list aiidalab-home

it will print the old version.

@danielhollas

Copy link
Copy Markdown
Contributor Author

@unkcpz would you mind giving me access to this repo as well? 😊

@unkcpz

unkcpz commented Nov 24, 2022

Copy link
Copy Markdown
Member

@unkcpz would you mind giving me access to this repo as well? 😊

Sure, done! You gradually have your hands on all repos 😄

@danielhollas danielhollas self-assigned this Nov 24, 2022

@unkcpz unkcpz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me. But why it is not updated? The variable is in the bumpver setup of setup.cfg.

@danielhollas

Copy link
Copy Markdown
Contributor Author

The last release was done by hand, the bumpver was introduced only in the last commit if you look at the git history.

@unkcpz

unkcpz commented Nov 24, 2022

Copy link
Copy Markdown
Member

I see, thanks for check!

@danielhollas

Copy link
Copy Markdown
Contributor Author

Btw: I caught this by writing a new test as part of aiidalab/aiidalab-docker-stack#327 :-)

@unkcpz unkcpz merged commit 1152ff7 into aiidalab:master Nov 24, 2022
@danielhollas danielhollas deleted the fix-version branch November 24, 2022 18:14
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.

2 participants