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 master build failing caused by newest humanize package missing VERSION attribute #1156

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

Tomasz-Kluczkowski
Copy link
Contributor

@Tomasz-Kluczkowski Tomasz-Kluczkowski commented Oct 16, 2021

When accessing humanize.VERSION we were running into an attribute error since they removed VERSION from that file.
We are supposed to use the standard __version__ now.

Also set humanize package version in stone.
Modify message from bugreport function.
Add test confirming new bugreport behaviour.

When accessing humanize.VERSION we were running into an attribute error since they removed VERSION from that file.
We are supposed to use the standard __version__ now.

Also set humanize package version in stone.
Modify message from bugreport function.
Add test confirming new bugreport behaviour.
@mher mher merged commit 70b1ae4 into mher:master Oct 19, 2021
pabs3 added a commit to pabs3/flower that referenced this pull request Mar 4, 2022
Also drop the humanize pinning since newer versions work too.

Fixes: commit 70b1ae4
See-also: mher#1156
pabs3 added a commit to pabs3/flower that referenced this pull request Mar 4, 2022
Also drop the humanize pinning since newer versions work too.

Fixes: commit 70b1ae4
See-also: mher#1156
pabs3 added a commit to pabs3/flower that referenced this pull request Mar 4, 2022
Also drop the humanize pinning since newer versions work too.

Fixes: commit 70b1ae4
See-also: mher#1156
mher pushed a commit that referenced this pull request May 21, 2022
Also drop the humanize pinning since newer versions work too.

Fixes: commit 70b1ae4
See-also: #1156
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