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(typescript): Always use ParsedCommandLine #162

Merged
merged 4 commits into from
Jan 23, 2020

Conversation

NotWoods
Copy link
Member

@NotWoods NotWoods commented Jan 13, 2020

Rollup Plugin Name: typescript

This PR contains:

  • bugfix
  • feature
  • refactor
  • documentation
  • other

Are tests included?

  • yes (bugfixes and features will not be merged without tests)
  • no

Breaking Changes?

  • yes (breaking changes will not be merged unless absolutely necessary)
  • no

List any relevant issue numbers:

Description

Refactors the option parsing to rely more on the Typescript API. ParsedCommandLine will later be used for plugin improvements as it includes additional data.

This is the first part of getting typechecking to work for the plugin. For that reason, we set up the fileNames array in this PR even though its not used yet.

packages/typescript/tsconfig.json Outdated Show resolved Hide resolved
NotWoods and others added 2 commits January 19, 2020 14:32
Co-Authored-By: Andrew Powell <shellscape@users.noreply.github.com>
@shellscape
Copy link
Collaborator

LGTM. Ship it.

@shellscape shellscape merged commit 8d35423 into rollup:master Jan 23, 2020
@NotWoods NotWoods deleted the ts-read-config branch January 25, 2020 18:51
LarsDenBakker pushed a commit to LarsDenBakker/plugins that referenced this pull request Sep 12, 2020
* chore(typescript): Always use ParsedCommandLine

* Update packages/typescript/tsconfig.json

Co-Authored-By: Andrew Powell <shellscape@users.noreply.github.com>

* Add default options and enforce noEmit

Co-authored-by: Andrew Powell <shellscape@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants