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

Fix version mismatch #121

Merged
merged 1 commit into from
Nov 24, 2022
Merged

Fix version mismatch #121

merged 1 commit into from
Nov 24, 2022

Conversation

danielhollas
Copy link
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
Contributor Author

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

@unkcpz
Copy link
Member

unkcpz commented Nov 24, 2022

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

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

Copy link
Member

@unkcpz unkcpz left a comment

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
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
Copy link
Member

unkcpz commented Nov 24, 2022

I see, thanks for check!

@danielhollas
Copy link
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