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

chore!: drop support for node.js 8 #554

Merged
merged 4 commits into from
Mar 26, 2020
Merged

chore!: drop support for node.js 8 #554

merged 4 commits into from
Mar 26, 2020

Conversation

JustinBeckwith
Copy link
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 25, 2020
@codecov
Copy link

codecov bot commented Mar 25, 2020

Codecov Report

Merging #554 into master will decrease coverage by 0.20%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #554      +/-   ##
==========================================
- Coverage   99.79%   99.59%   -0.21%     
==========================================
  Files           5        5              
  Lines        1955     1953       -2     
  Branches      172      171       -1     
==========================================
- Hits         1951     1945       -6     
- Misses          4        7       +3     
- Partials        0        1       +1     
Impacted Files Coverage Δ
src/index.ts 100.00% <100.00%> (ø)
src/operation.ts 100.00% <100.00%> (ø)
src/service-object.ts 100.00% <100.00%> (ø)
src/service.ts 99.22% <100.00%> (-0.39%) ⬇️
src/util.ts 99.32% <100.00%> (-0.34%) ⬇️

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 63175e0...848ce25. Read the comment docs.

@bcoe
Copy link
Contributor

bcoe commented Mar 26, 2020

The Windows CI/CD issue is most likely due to linting being run as a post test step.

We might want to dig into why linting has issues on Windows ... but for now, it might be okay to just drop that post test check.

@bcoe bcoe merged commit 9f41047 into master Mar 26, 2020
@bcoe bcoe deleted the no8 branch March 26, 2020 19:11
@release-please release-please bot mentioned this pull request Mar 26, 2020
yoshi-automation added a commit that referenced this pull request Apr 1, 2020
9f41047
commit 9f41047
Author: Justin Beckwith <justin.beckwith@gmail.com>
Date:   Thu Mar 26 12:11:44 2020 -0700

    chore!: drop support for node.js 8 (#554)
alexander-fenster pushed a commit that referenced this pull request Apr 1, 2020
* Change triggered by none of the following:
This git repo (https://github.com/googleapis/nodejs-common.git)
Git repo https://github.com/googleapis/synthtool.git

* fix(deps): update dependency @google-cloud/projectify to v2 (#553)

23030a2
commit 23030a2
Author: WhiteSource Renovate <bot@renovateapp.com>
Date:   Wed Mar 25 22:16:03 2020 +0100

    fix(deps): update dependency @google-cloud/projectify to v2 (#553)

* fix(deps): update dependency @google-cloud/promisify to v2 (#552)

63175e0
commit 63175e0
Author: WhiteSource Renovate <bot@renovateapp.com>
Date:   Wed Mar 25 22:35:58 2020 +0100

    fix(deps): update dependency @google-cloud/promisify to v2 (#552)

* chore!: drop support for node.js 8 (#554)

9f41047
commit 9f41047
Author: Justin Beckwith <justin.beckwith@gmail.com>
Date:   Thu Mar 26 12:11:44 2020 -0700

    chore!: drop support for node.js 8 (#554)

* chore: release 3.0.0 (#542)

13ae85c
commit 13ae85c
Author: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Date:   Fri Mar 27 12:51:09 2020 -0700

    chore: release 3.0.0 (#542)

* fix(deps): update dependency google-auth-library to v6 (#556)

03a8a54
commit 03a8a54
Author: WhiteSource Renovate <bot@renovateapp.com>
Date:   Fri Mar 27 21:12:35 2020 +0100

    fix(deps): update dependency google-auth-library to v6 (#556)

* build: set AUTOSYNTH_MULTIPLE_COMMITS=true for context aware commits (#559)

575353f
commit 575353f
Author: Benjamin E. Coe <bencoe@google.com>
Date:   Tue Mar 31 18:35:28 2020 -0700

    build: set AUTOSYNTH_MULTIPLE_COMMITS=true for context aware commits (#559)

* docs: document the release schedule we follow (#454)

googleapis/synthtool@6a17abc
commit 6a17abc7652e2fe563e1288c6e8c23fc260dda97
Author: Benjamin E. Coe <bencoe@google.com>
Date:   Mon Mar 23 19:22:34 2020 -0700

    docs: document the release schedule we follow (#454)

* fix: do not run node 8 CI (#456)

googleapis/synthtool@1b4cc80
commit 1b4cc80a7aaf164f6241937dd87f3bd1f4149e0c
Author: Alexander Fenster <fenster@google.com>
Date:   Wed Mar 25 08:01:31 2020 -0700

    fix: do not run node 8 CI (#456)

* fix: update template files for Node.js libraries (#463)

googleapis/synthtool@9982024
commit 99820243d348191bc9c634f2b48ddf65096285ed
Author: Alexander Fenster <fenster@google.com>
Date:   Tue Mar 31 11:56:27 2020 -0700

    fix: update template files for Node.js libraries (#463)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants