Skip to content

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

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