Skip to content

build!: update to latest version of gts/typescript#183

Merged
bcoe merged 3 commits intomasterfrom
update-gts
Mar 23, 2020
Merged

build!: update to latest version of gts/typescript#183
bcoe merged 3 commits intomasterfrom
update-gts

Conversation

@bcoe
Copy link

@bcoe bcoe commented Mar 23, 2020

BREAKING CHANGE: typescript@3.7 introduced some breaking changes

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 23, 2020
@@ -1,3 +1,5 @@
/* eslint-disable prefer-rest-params */
Copy link
Contributor

Choose a reason for hiding this comment

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

We should move this after the license header I think

Copy link
Author

Choose a reason for hiding this comment

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

removed the eslint-disable inline statements, we will probably want a replacement soonish depending on how migrating other libraries goes.

@@ -66,8 +68,7 @@ export function promisify(
const slice = Array.prototype.slice;

// tslint:disable-next-line:no-any
Copy link
Contributor

Choose a reason for hiding this comment

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

Sadly these won't work with eslint. There's a different syntax (which I can't remember right now) for disabling a line.

Copy link
Contributor

Choose a reason for hiding this comment

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

Screen Shot 2020-03-23 at 1 47 33 PM

Is this useful 👆

bcoe added 3 commits March 23, 2020 10:55
BREAKING CHANGE: typescript@3.7 introduced some breaking changes
@codecov
Copy link

codecov bot commented Mar 23, 2020

Codecov Report

Merging #183 into master will not change coverage by %.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #183   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines          220       219    -1     
  Branches        37        36    -1     
=========================================
- Hits           220       219    -1     
Impacted Files Coverage Δ
src/index.ts 100.00% <100.00%> (ø)

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 7b703ce...4c5ae85. Read the comment docs.

@bcoe bcoe merged commit 9c3ed12 into master Mar 23, 2020
@bcoe bcoe deleted the update-gts branch March 23, 2020 18:52
@release-please release-please bot mentioned this pull request Mar 23, 2020
gcf-merge-on-green bot pushed a commit that referenced this pull request Mar 23, 2020
🤖 I have created a release \*beep\* \*boop\* 
---
## [2.0.0](https://www.github.com/googleapis/nodejs-promisify/compare/v1.0.4...v2.0.0) (2020-03-23)


### ⚠ BREAKING CHANGES

* update to latest version of gts/typescript (#183)
* drop Node 8 from engines field (#184)

### Features

* drop Node 8 from engines field ([#184](https://www.github.com/googleapis/nodejs-promisify/issues/184)) ([7e6d3c5](https://www.github.com/googleapis/nodejs-promisify/commit/7e6d3c54066d89530ed25c7f9722efd252f43fb8))


### Build System

* update to latest version of gts/typescript ([#183](https://www.github.com/googleapis/nodejs-promisify/issues/183)) ([9c3ed12](https://www.github.com/googleapis/nodejs-promisify/commit/9c3ed12c12f4bb1e17af7440c6371c4cefddcd59))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please).
yoshi-automation added a commit that referenced this pull request Apr 1, 2020
9c3ed12
commit 9c3ed12
Author: Benjamin E. Coe <bencoe@google.com>
Date:   Mon Mar 23 11:51:57 2020 -0700

    build!: update to latest version of gts/typescript (#183)
yoshi-automation added a commit that referenced this pull request Apr 1, 2020
9f530f8
commit 9f530f8
Author: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Date:   Mon Mar 23 18:58:21 2020 +0000

    chore: release 2.0.0 (#185)

    🤖 I have created a release \*beep\* \*boop\*
    ---
    ## [2.0.0](https://www.github.com/googleapis/nodejs-promisify/compare/v1.0.4...v2.0.0) (2020-03-23)

    ### ⚠ BREAKING CHANGES

    * update to latest version of gts/typescript (#183)
    * drop Node 8 from engines field (#184)

    ### Features

    * drop Node 8 from engines field ([#184](https://www.github.com/googleapis/nodejs-promisify/issues/184)) ([7e6d3c5](https://www.github.com/googleapis/nodejs-promisify/commit/7e6d3c54066d89530ed25c7f9722efd252f43fb8))

    ### Build System

    * update to latest version of gts/typescript ([#183](https://www.github.com/googleapis/nodejs-promisify/issues/183)) ([9c3ed12](https://www.github.com/googleapis/nodejs-promisify/commit/9c3ed12c12f4bb1e17af7440c6371c4cefddcd59))
    ---

    This PR was generated with [Release Please](https://github.com/googleapis/release-please).
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