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 prerequisites.sh #2873

Merged
merged 5 commits into from
Apr 12, 2021
Merged

Fix prerequisites.sh #2873

merged 5 commits into from
Apr 12, 2021

Conversation

catalinnnn
Copy link
Contributor

In case PRIVATE_REPOS is not defined, the script will exit with error. Using double brackets will take care of this case.

In case PRIVATE_REPOS is not defined, the script will exit with error. Using double brackets will take care of this case.
@codecov
Copy link

codecov bot commented Mar 2, 2021

Codecov Report

Merging #2873 (6b697d6) into master (2d8e5c4) will not change coverage.
The diff coverage is n/a.

❗ Current head 6b697d6 differs from pull request most recent head e79319c. Consider uploading reports for the commit e79319c to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2873   +/-   ##
=======================================
  Coverage   74.81%   74.81%           
=======================================
  Files         613      613           
  Lines       58512    58512           
=======================================
  Hits        43776    43776           
  Misses      10801    10801           
  Partials     3935     3935           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2d8e5c4...e79319c. Read the comment docs.

@@ -56,7 +56,7 @@ fi
cd $(dirname $ELRONDDIR)
git clone git@github.com:ElrondNetwork/elrond-deploy-go.git

if [ $PRIVATE_REPOS -eq 1 ]; then
if [[ $PRIVATE_REPOS -eq 1 ]]; then
git clone git@github.com:ElrondNetwork/elrond-proxy-go.git
Copy link
Contributor

Choose a reason for hiding this comment

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

can you please take out this line outside of this if branch? elrond-proxy-go is no longer a private repo :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure :) done

iulianpascalau
iulianpascalau previously approved these changes Mar 3, 2021
sasurobert
sasurobert previously approved these changes Mar 23, 2021
@iulianpascalau iulianpascalau changed the base branch from master to external April 12, 2021 08:57
@iulianpascalau iulianpascalau dismissed stale reviews from sasurobert and themself April 12, 2021 08:57

The base branch was changed.

@iulianpascalau iulianpascalau merged commit 4f48f01 into multiversx:external Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants