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

Release script for adapters #120

Merged
merged 2 commits into from
May 19, 2017
Merged

Release script for adapters #120

merged 2 commits into from
May 19, 2017

Conversation

beeender
Copy link
Contributor

No description provided.

Copy link
Contributor

@bmeike bmeike left a comment

Choose a reason for hiding this comment

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

just checking on the git clean -xfd. Might want to warn or something...

git fetch --all
git checkout releases
git reset --hard origin/releases
git clean -xfd
Copy link
Contributor

Choose a reason for hiding this comment

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

This will remove all .idea, *.iml, and local.properties files. Is that ok?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think so. And I really hope people release from a dedicated project instead of the daily working project directory. Make sure the project dir is clean is very much the most important thing for release.

tools/release.sh Outdated

# Script to make release on the local machine.
# See https://github.com/realm/realm-wiki/wiki/Java-Release-Checklist for more details.
# FIXME: Only patch release is supported now.
Copy link
Contributor

Choose a reason for hiding this comment

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

remove this

@beeender beeender merged commit 87610d1 into releases May 19, 2017
@beeender beeender deleted the mc/release-script branch May 19, 2017 02:09
@beeender beeender removed the S:Review label May 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants