Skip to content

chore: upgrade circleci config to v2.1 #6312

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

Merged
merged 4 commits into from
Aug 26, 2019
Merged

Conversation

pimlie
Copy link

@pimlie pimlie commented Aug 26, 2019

Types of changes

  • Bug fix (a non-breaking change which fixes an issue)
  • New feature (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

This upgrades the current circleci config to v2.1. One of the main benefits of v2.1 is that builds can be automatically cancelled when a new build is triggered, see here for more info. This could save some time in some cases.

This config is based on the config I have been using for nuxt/vue-meta and nuxt/press

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly. (PR: #)
  • I have added tests to cover my changes (if not applicable, please state why)
  • All new and existing tests are passing.

Sorry, something went wrong.

@pimlie pimlie requested review from clarkdo, pi0 and a team August 26, 2019 15:44
@pimlie
Copy link
Author

pimlie commented Aug 26, 2019

Btw, the tests for this PR are still using the old config

--edit--

Fyi, i checked the config with the cli tool:

nuxt.js# circleci config validate
Config file at .circleci/config.yml is valid.

@codecov-io
Copy link

codecov-io commented Aug 26, 2019

Codecov Report

Merging #6312 into dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##             dev   #6312   +/-   ##
=====================================
  Coverage   95.7%   95.7%           
=====================================
  Files         79      79           
  Lines       2652    2652           
  Branches     683     683           
=====================================
  Hits        2538    2538           
  Misses        98      98           
  Partials      16      16
Flag Coverage Δ
#e2e 100% <ø> (ø) ⬆️
#fixtures 50.22% <ø> (ø) ⬆️
#unit 92.3% <ø> (ø) ⬆️

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 f844eef...bb3642d. Read the comment docs.

@pimlie
Copy link
Author

pimlie commented Aug 26, 2019

@clarkdo Feel free to make changes to this pr yourself if you really believe this is a blocking issue

@clarkdo
Copy link
Member

clarkdo commented Aug 26, 2019

I don’t think it anything is blocked, it’s just in reviewing, we can hear feedback from others, if no concerns, it can be merged

@galvez
Copy link

galvez commented Aug 26, 2019

@clarkdo @pimlie Despite the annoying syntax, I tend to think the boolean set of attributes is better, so no objections from me in merging the PR as-is.

@galvez galvez requested a review from TheAlexLichter August 26, 2019 21:56
@clarkdo
Copy link
Member

clarkdo commented Aug 26, 2019

Cool

@clarkdo clarkdo merged commit c4c51e3 into nuxt:dev Aug 26, 2019
@pimlie pimlie deleted the use-circleci-v2.1 branch August 27, 2019 03:57
@pimlie
Copy link
Author

pimlie commented Aug 27, 2019

@clarkdo Thank you for merging 🤗

@pi0 Pinging you for visibility so you know circleci pipelines are now enabled

atinux pushed a commit that referenced this pull request Aug 28, 2019
* [release]

* fix(vue-app): enforce default css when used with frameworks (#6255)

* chore(deps): update dependency serialize-javascript to ^1.8.0 (#6254)

* fix(config): provide typescript backward compatibility (#6258)

* fix(vue-app): only overwrite leave listener when none provided or without done arg (#6264)

* fix(nuxt-start): add missing `vue-client-only` dependency (#6267)

* fix(server): treat `https: null` as `https: undefined` (#6265)

* chore(deps): update devdependency rollup to ^1.20.0 (#6268)

* fix: improve watching experience for generated files (#6257)

* refactor(core): use hable (#6271)

* test: disable terser/minify by default (#6290)

* chore(deps): update devdependency rollup to ^1.20.1 (#6282)

* chore(cli): accept hooks (#6274)

* chore(deps): update devdependency rollup-plugin-alias to v2 (#6281)

* fix(builder): apply overrides from app dir only (#6283)

[release]

* feat: function watchQuery (#6245)

* Revert "feat: function watchQuery (#6245)" (#6296)

This reverts commit 3c61830.

* chore(deps): update devdependency rollup to ^1.20.2 (#6295)



Co-authored-by: Renovate Bot <bot@renovateapp.com>

* test(vue-app): add template compiler helper (#6299)

* test: add jest roots for less greedy test search (#6300)

* test: remove unnecessary generate (#6301)

* fix: update the minimal require node version in distributions (#6310)

* chore(deps): update devdependency babel-eslint to ^10.0.3 (#6304)

* chore(deps): lock file maintenance (#6305)

* chore: upgrade circleci config to v2.1 (#6312)

* test: add test for modern bundle size (#6302)

* test: add client-only test to basic fixture (#6315)

test: check for no-ssr deprecation warning
@danielroe danielroe added the 2.x label Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants