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(deploy): use javascript semantic-release package for deployment #126

Merged
merged 2 commits into from
Mar 20, 2019

Conversation

AntoineDao
Copy link
Member

Using the "pure" javascript implementation of semantic-release makes it a bit more flexible to set
up custom scripts to run at different stages of the semantic release process. The packages used as
well as scripts to run at different stages are specified in the .releaserc.json file. In this case
we use standard semantic-release and add a custom deploy script on top of the normal github release
setup. The custom deploy script is the new 'deploy.sh' file which takes the new version number as an
argument to deploy the package to PyPi and build the documentation.

#125

Using the "pure" javascript implementation of semantic-release makes it a bit more flexible to set
up custom scripts to run at different stages of the semantic release process. The packages used as
well as scripts to run at different stages are specified in the .releaserc.json file. In this case
we use standard semantic-release and add a custom deploy script on top of the normal github release
setup. The custom deploy script is the new 'deploy.sh' file which takes the new version number as an
argument to deploy the package to PyPi and build the documentation.

ladybug-tools#125

build(pytest): add pytest-cov deps to dev-requirements.txt

build(euclid): add euclid to dev-requirements.txt
@AntoineDao
Copy link
Member Author

Finally made it after so many force pushes to get dev-requirements.txt to play ball!

Copy link
Member

@mostaphaRoudsari mostaphaRoudsari left a comment

Choose a reason for hiding this comment

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

Hi @AntoineDao, Thank you for getting this to work. I couldn't see from the code how you pass the version is the new workflow but I think I will know when I see it. Let's merge this in and see how it will work when merged to master.

@mostaphaRoudsari mostaphaRoudsari added enhancement New feature or request documentation For issues requiring changes to documentation labels Mar 20, 2019
@AntoineDao AntoineDao merged commit f6701b9 into ladybug-tools:master Mar 20, 2019
@ladybugbot
Copy link
Contributor

🎉 This PR is included in version 0.3.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation For issues requiring changes to documentation enhancement New feature or request released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants