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

Simplify Travis workflow #1732

Merged
merged 1 commit into from
Jun 9, 2020
Merged

Simplify Travis workflow #1732

merged 1 commit into from
Jun 9, 2020

Conversation

KtorZ
Copy link
Member

@KtorZ KtorZ commented Jun 9, 2020

Issue Number

N/A

Overview

  • removed the haddock export, this is taking a huge time and has become a pain in the *** for a very small added value. It can be moved later to a nightly if really needed.
  • Still fetches the artifacts from Hydra, this should now be quite quick.
  • And, still automatically export the API documentation from the swagger file, latest and pinned versions on releases.

Comments

- removed the haddock export, this is taking a huge time and has become a pain in the *** for a very small added value. It can be moved later to a nightly if really needed.
- Still fetches the artifacts from Hydra, this should now be quite quick.
- And, still automatically export the API documentation from the swagger file, latest and pinned versions on releases.
@KtorZ KtorZ requested a review from Anviking June 9, 2020 09:06
@KtorZ KtorZ self-assigned this Jun 9, 2020
- git fetch origin
- git checkout gh-pages && git merge -X theirs --no-commit --no-ff --allow-unrelated-histories - && git commit --allow-empty --no-edit
- git push -f -q https://WilliamKingNoel-Bot:[email protected]/input-output-hk/cardano-wallet gh-pages &>/dev/null

- stage: deploy 🚀
if: type != pull_request AND (tag =~ ^v OR commit_message =~ "TRAVIS_TRIGGER_RELEASE")
if: type != pull_request AND (tag =~ ^v OR commit_message =~ /TRAVIS_TRIGGER_RELEASE/)
Copy link
Member Author

Choose a reason for hiding this comment

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

Noticed it when I added the PRE_CACHE mark, =~ actually expects a regexp.. so this would have never worked.

@Anviking
Copy link
Member

Anviking commented Jun 9, 2020

bors r+

iohk-bors bot added a commit that referenced this pull request Jun 9, 2020
1732: Simplify Travis workflow r=Anviking a=KtorZ

# Issue Number

<!-- Put here a reference to the issue this PR relates to and which requirements it tackles -->

N/A

# Overview

<!-- Detail in a few bullet points the work accomplished in this PR -->

- removed the haddock export, this is taking a huge time and has become a pain in the *** for a very small added value. It can be moved later to a nightly if really needed.
- Still fetches the artifacts from Hydra, this should now be quite quick.
- And, still automatically export the API documentation from the swagger file, latest and pinned versions on releases.


# Comments

<!-- Additional comments or screenshots to attach if any -->

<!-- 
Don't forget to:

 ✓ Self-review your changes to make sure nothing unexpected slipped through
 ✓ Assign yourself to the PR
 ✓ Assign one or several reviewer(s)
 ✓ Once created, link this PR to its corresponding ticket
 ✓ Assign the PR to a corresponding milestone
 ✓ Acknowledge any changes required to the Wiki
-->


Co-authored-by: KtorZ <[email protected]>
@KtorZ KtorZ merged commit 5290acf into master Jun 9, 2020
@KtorZ KtorZ deleted the KtorZ/simplify-travis branch June 9, 2020 09:21
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.

2 participants