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: improve TS automated type tests #6860

Merged
merged 2 commits into from
Feb 11, 2021
Merged

chore: improve TS automated type tests #6860

merged 2 commits into from
Feb 11, 2021

Conversation

jackfranklin
Copy link
Collaborator

This PR:

  1. Makes sure we remove and freshly install Puppeteer before testing our
    type defs, to avoid running on stale files.
  2. Makes the tests run off puppeteer.tgz to avoid having version
    numbers in the file name and therefore having to update it when we
    bump versions.

Copy link
Member

@mathiasbynens mathiasbynens left a comment

Choose a reason for hiding this comment

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

Nice! Just a suggestion and a question

scripts/test-ts-definition-files.ts Outdated Show resolved Hide resolved
This PR:

1. Makes sure we remove and freshly install Puppeteer before testing our
   type defs, to avoid running on stale files.
2. Makes the tests run off `puppeteer.tgz` to avoid having version
   numbers in the file name and therefore having to update it when we
   bump versions.
@jackfranklin jackfranklin enabled auto-merge (squash) February 11, 2021 10:22
@jackfranklin jackfranklin merged commit 641ffc2 into main Feb 11, 2021
@jackfranklin jackfranklin deleted the improve-ts-test branch February 11, 2021 10:34
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

2 participants