Skip to content
This repository has been archived by the owner on Jun 17, 2024. It is now read-only.

fix: remove deploy step #22

Merged
merged 2 commits into from
Jun 5, 2020
Merged

fix: remove deploy step #22

merged 2 commits into from
Jun 5, 2020

Conversation

elizabethsjudd
Copy link
Contributor

@elizabethsjudd elizabethsjudd commented Jun 5, 2020 •
edited
Loading

Changelog

Summary

Deploy is creating a lot of weird issues so trying a conditional build step instead

Updated

  • travis to use a conditional step instead of deploy
  • removed previous fix attempts

Definition of done

  • The code has been reviewed by the Maintainer
  • The code has passed quality checks
    • 85% code coverage
    • Passed the following tests:
      • Unit
  • Documents are updated

elizabethsjudd
elizabethsjudd commented Jun 5, 2020
View reviewed changes
.travis.yml Outdated Show resolved Hide resolved
@elizabethsjudd elizabethsjudd merged commit 536c7e3 into IBM:stable Jun 5, 2020
@elizabethsjudd elizabethsjudd deleted the travis-update branch June 5, 2020 16:42
@elizabethsjudd
Copy link
Contributor Author

🎉 This PR is included in version 1.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@scottnath scottnath scottnath approved these changes

Assignees
No one assigned
Labels
released on @latest
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@elizabethsjudd @scottnath