Skip to content
This repository was archived by the owner on May 4, 2019. It is now read-only.

Swap Travis for Circle CI #188

Merged
merged 20 commits into from
Jul 2, 2018
Merged

Swap Travis for Circle CI #188

merged 20 commits into from
Jul 2, 2018

Conversation

puregarlic
Copy link
Contributor

@puregarlic puregarlic commented Jun 29, 2018

Updates Made

Circle CI is a far more extensible and performant CI solution than Travis, so this PR has replicated the current Travis configuration for Circle. Additionally, it fixes some untested and most-likely broken functionality (like promoting the beta tag to latest). Additionally, I've added JUnit reporting so we can see failed tests at-a-glance.

Here's what the pipeline looks like:

+-------------+
|Checkout code|
+------+------+
       |
       |
       |
+------v-----------------+
|Install NPM dependencies+--+
+------+-----------------+  |
       |                    |
       |                    |
       |                    |
+------v-------+    +-------v-------+
|Run unit tests|    |Lint Typescript|
+------+-------+    +---------------+
       |
       +--------------------+--------------------+
       |                    |                    |
+------v--------+    +------v------+    +--------v--------+
|Build storybook|    |Build typedoc|    |Run webpack build|
+------+--------+    +------+------+    +-----------------+
       |                    |
       |                    |
       |                    |
+------v----+               |
|Deploy docs<---------------+
+---+-------+
    |
    |
    |
+---v---+
|Release|
+-------+

Contributing developer checklist

  • I've updated source branch with the latest changes from dev.
  • I've added/changed unit tests for components with functionality.
  • I've added/updated storybook for any component that I've changed.
  • I've reviewed each jsdoc header in regards to the changes that I've made and updated them.

Reviewing developer checklist

  • I've pulled this branch to my local machine.
  • I've inspected the changes on storybook.
  • I've run the test suite and all tests have passed

@puregarlic puregarlic requested a review from tnoelcke June 29, 2018 22:36
@puregarlic puregarlic self-assigned this Jun 29, 2018
@puregarlic
Copy link
Contributor Author

Before this pipeline works completely, we need to establish GH_EMAIL and GH_NAME environmental variables. These are necessary for publishing documentation, but since we haven't published to master for awhile, it would be easy to forget about this.

@tnoelcke
Copy link

tnoelcke commented Jul 2, 2018

Lets make sure we get that done before I merge this pull request so it doesn't get forgotten.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.091% when pulling 90a9134 on setup/circleci into b08bbf6 on dev.

@tnoelcke tnoelcke merged commit 53dd52e into dev Jul 2, 2018
@cass-osu
Copy link

cass-osu commented Jul 5, 2018

🎉 This PR is included in version 1.15.0 🎉

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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants