Skip to content
This repository has been archived by the owner on Aug 22, 2023. It is now read-only.

chore: update circle config #21

Merged
merged 1 commit into from
Oct 4, 2019
Merged

chore: update circle config #21

merged 1 commit into from
Oct 4, 2019

Conversation

RasPhilCo
Copy link
Contributor

No description provided.

@RasPhilCo RasPhilCo requested a review from chadian October 3, 2019 23:54
@codecov
Copy link

codecov bot commented Oct 3, 2019

Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   70.54%   70.54%           
=======================================
  Files           7        7           
  Lines         129      129           
  Branches       31       31           
=======================================
  Hits           91       91           
  Misses         27       27           
  Partials       11       11

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 bb3c631...39543c5. Read the comment docs.

@@ -50,7 +50,8 @@ workflows:
"@oclif/errors":
jobs:
- node-latest
- node-8
- node-12
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it also needs node-12 and node-10 to extend off of test.

  node-10:
    <<: *test
    docker:
      - image: node:10

Copy link
Contributor Author

Choose a reason for hiding this comment

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

^that's defined above, these are the workflows declaration

Copy link
Contributor

Choose a reason for hiding this comment

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

Ahh, my bad. Looks good.

Copy link
Contributor

@chadian chadian left a comment

Choose a reason for hiding this comment

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

🤙🏽

@RasPhilCo RasPhilCo merged commit 66f6cfa into master Oct 4, 2019
@RasPhilCo RasPhilCo deleted the fix-circle branch October 4, 2019 18:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants