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

Website update release script fixes #207

Merged
merged 4 commits into from Jul 19, 2020
Merged

Conversation

@rcurtin
Copy link
Member

@rcurtin rcurtin commented Jul 18, 2020

Everything worked pretty well (but not perfectly) for the automatic 2.13.0 release. This fixes some simple bugs I found in the website update script.

rcurtin added 2 commits Jul 18, 2020
@zoq
zoq approved these changes Jul 18, 2020
Copy link
Member

@zoq zoq left a comment

Looks good to me.

@mlpack-bot
mlpack-bot bot approved these changes Jul 19, 2020
Copy link

@mlpack-bot mlpack-bot bot left a comment

Second approval provided automatically after 24 hours. 👍

@rcurtin rcurtin merged commit 852de21 into mlpack:master Jul 19, 2020
4 of 5 checks passed
4 of 5 checks passed
Memory Checks Build finished.
Details
Static Code Analysis Checks Build finished.
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
mlpack master build test Build finished.
Details
@rcurtin rcurtin deleted the rcurtin:release-script-fixes branch Jul 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.