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

chore: Fix the release script #2325

Merged
merged 5 commits into from
Jul 27, 2019
Merged

chore: Fix the release script #2325

merged 5 commits into from
Jul 27, 2019

Conversation

atm1504
Copy link
Member

@atm1504 atm1504 commented Jul 26, 2019

Fixes #2324

Changes:
Updated the script according to the one present in : fossasia/open-event-organizer-android#1770

@auto-label auto-label bot added the chore label Jul 26, 2019
build.gradle Outdated

ext {
KEYSTORE_FILE = rootProject.file('scripts/key.jks')
CIRCLE_BUILD = System.getenv("CIRCLE") == "true" && KEYSTORE_FILE.exists()
Copy link
Member

Choose a reason for hiding this comment

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

Show me where does it say that CircleCI Builds have CIRCLE as an environment variable

Copy link
Member Author

Choose a reason for hiding this comment

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

Changed it to CIRCLECI. This is present in the documentation.

@iamareebjamal
Copy link
Member

You also have to run prep-key before the build. Please see the reference PR carefully

@atm1504
Copy link
Member Author

atm1504 commented Jul 26, 2019

You also have to run prep-key before the build. Please see the reference PR carefully

Updated the circleci script. prep-key is now executed before ./gradlew build

@ci-reporter
Copy link

ci-reporter bot commented Jul 26, 2019

The build is failing

✨ Good work on this PR so far! ✨ Unfortunately, the Circle CI build is failing as of e2db68b. Here's the output:

Run Tests
> Task :app:preBuild UP-TO-DATE

I'm sure you can fix it! If you need help, don't hesitate to ask a maintainer of the project!


This comment was automagically generated by ci-reporter. If you see a problem, open an issue here.

1 similar comment
@ci-reporter
Copy link

ci-reporter bot commented Jul 26, 2019

The build is failing

✨ Good work on this PR so far! ✨ Unfortunately, the Circle CI build is failing as of e2db68b. Here's the output:

Run Tests
> Task :app:preBuild UP-TO-DATE

I'm sure you can fix it! If you need help, don't hesitate to ask a maintainer of the project!


This comment was automagically generated by ci-reporter. If you see a problem, open an issue here.

@iamareebjamal iamareebjamal changed the base branch from development to master July 27, 2019 00:42
@iamareebjamal iamareebjamal merged commit cb95f99 into fossasia:master Jul 27, 2019

ext {
KEYSTORE_FILE = rootProject.file('scripts/key.jks')
Copy link
Member

Choose a reason for hiding this comment

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

This should have been exec/key.jks

Copy link
Member Author

Choose a reason for hiding this comment

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

In the open event repo, there is a folder called exec, inside which all the files were kept. But here we have a folder named scripts, where such files are kept. So I used scripts there.

Can you please tell what's the use of the above statement.

Copy link
Member

Choose a reason for hiding this comment

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

You are saying completely opposite things. open event repo has scripts folder and that's why scripts was used and susi has exec folder. You didn't change anything. You just copied. You should have changed this to exec

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants