Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Update gitlab-test.sh #7883

Merged
merged 2 commits into from
Feb 13, 2018
Merged

Update gitlab-test.sh #7883

merged 2 commits into from
Feb 13, 2018

Conversation

General-Beck
Copy link
Contributor

@General-Beck General-Beck added A0-pleasereview 🤓 Pull request needs code review. B0-patch labels Feb 13, 2018
fi
if [[ "$CI_COMMIT_REF_NAME" == "nightly" ]];
then sh scripts/aura-test.sh;
then sh scripts/aura-test.sh || exit $?;
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think scritps/aura-test.sh should also have set -eu on top.

@5chdn 5chdn added the M1-ci 🙉 Continuous integration. label Feb 13, 2018
@5chdn 5chdn added this to the 1.10 milestone Feb 13, 2018
@5chdn 5chdn merged commit bf7e874 into master Feb 13, 2018
@5chdn 5chdn deleted the gitlab-fix-test branch February 13, 2018 15:21
5chdn pushed a commit that referenced this pull request Feb 14, 2018
* Update gitlab-test.sh

#7871

* Update aura-test.sh
5chdn pushed a commit that referenced this pull request Feb 14, 2018
* Update gitlab-test.sh

#7871

* Update aura-test.sh
5chdn added a commit that referenced this pull request Feb 14, 2018
* Resolve conflicts

* Resolve conflicts

* Update gitlab-build.sh (#7855)

fix build ```version``` after #7723

* Resolve conflicts

* Update gitlab-test.sh (#7883)

* Update gitlab-test.sh

#7871

* Update aura-test.sh

* Backport master ci prs

* Restore js-build

* Bump stable to 1.8.10

* Bump stable to 1.8.10

* Make track stable
5chdn added a commit that referenced this pull request Feb 14, 2018
* Resolve conflicts

* Resolve conflicts

* Update gitlab-build.sh (#7855)

fix build ```version``` after #7723

* Resolve conflicts

* Update gitlab-test.sh (#7883)

* Update gitlab-test.sh

#7871

* Update aura-test.sh

* Backport master ci prs

* Bump beta to 1.9.3

* Make track beta

* Downgrade rustc_version
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-pleasereview 🤓 Pull request needs code review. M1-ci 🙉 Continuous integration.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants