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
Update changelog for 0.33.0 #594
Conversation
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.
Looks good, couple of things to resolve then can be merged. I have this locally:
Commits:
* PR #521: Allow instructions to be removed from blocks
* PR #548: Upgrade to LLVM9
* PR #575: Update the release date
* PR #577: llvmlite wheel building fixes
* PR #580: Trove classifiers may be out of date.
* PR #581: Add FAQ entry on LLVM version support.
* PR #582: Adds override for LLVM version check, re-formats docs.
* PR #586: Update changelog for 0.32.1 on master
* PR #590: add python_requires to setup.py
* PR #591: buildscripts: Unpin wheel
* PR #592: Fix meta.yaml
* PR #593: Fix CUDA with LLVM9
Total PRs: 12
CHANGE_LOG
Outdated
* PR #586: Update changelog for 0.32.1 on master | ||
* PR #585: Update changelog for 0.32.1 on release branch |
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.
Do these need to go in ? I removed similar from the Numba change log.
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.
right, they don't need to
CHANGE_LOG
Outdated
* PR #582: Adds override for LLVM version check, re-formats docs. | ||
* PR #581: Add FAQ entry on LLVM version support. | ||
* PR #577: llvmlite wheel building fixes | ||
* PR #576: Update the release date |
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.
Is this on master?
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.
good catch.
CHANGE_LOG
Outdated
* PR #590: add python_requires to setup.py | ||
* PR #586: Update changelog for 0.32.1 on master | ||
* PR #585: Update changelog for 0.32.1 on release branch | ||
* PR #580: Trove classifiers may be out of date. |
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.
This should be 2 lines down, after #581
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 my script is going by some sort of dates
as per review
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.
Thanks for the fixes.
CHANGE_LOG
Outdated
@@ -1,7 +1,30 @@ | |||
v0.33.0dev | |||
v0.33.0rc1 |
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 think this has the proposed release date in it now and no rc1
?
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.
Approved again post updating with release date.
as titled