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

fix: drop support for node.js 4.x and 9.x #97

Merged
merged 3 commits into from
Jun 12, 2018
Merged

fix: drop support for node.js 4.x and 9.x #97

merged 3 commits into from
Jun 12, 2018

Conversation

JustinBeckwith
Copy link
Contributor

BREAKING CHANGE: Removes Node.js 4.x and 9.x from the CI. Resolves #64.

@JustinBeckwith JustinBeckwith requested review from alexander-fenster and a team June 12, 2018 04:31
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 12, 2018
@ghost ghost assigned JustinBeckwith Jun 12, 2018
@codecov
Copy link

codecov bot commented Jun 12, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #97   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          19     19           
  Lines        1581   1581           
=====================================
  Hits         1581   1581

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 53d3699...3dcf534. Read the comment docs.

@fhinkel
Copy link
Contributor

fhinkel commented Jun 12, 2018

Very much in favor of this. Do we need to change the version of the library or do something else special?

@JustinBeckwith
Copy link
Contributor Author

Nope! We are working on multiple breaking changes right now (see #99). After they're all in, we will bump the version in it's own PR and ship a new release.

@JustinBeckwith JustinBeckwith merged commit 233c58e into googleapis:master Jun 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants