Skip to content

fix(typescript): add composite to validation checks#618

Merged
NotWoods merged 5 commits intorollup:masterfrom
dummdidumm:patch-1
Oct 23, 2020
Merged

fix(typescript): add composite to validation checks#618
NotWoods merged 5 commits intorollup:masterfrom
dummdidumm:patch-1

Conversation

@dummdidumm
Copy link
Contributor

@dummdidumm dummdidumm commented Oct 23, 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

TypeScript's composite compiler option produces declaration files, too. I stumbled upon this and did get a generic validation error right at the end when the plugin wants to print the files to the outdir but cannot find one. To fix it, I added an additional check to the relevant validation.

TypeScript's `composite` compiler option produces declaration files, too. Add it to the relevant validation check.
Was not there before, too, seems to need more setup
@dummdidumm dummdidumm marked this pull request as ready for review October 23, 2020 07:20
Copy link
Member

@NotWoods NotWoods left a comment

Choose a reason for hiding this comment

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

LGTM!

@NotWoods NotWoods changed the title fix(typescript) add composite to validation checks fix(typescript): add composite to validation checks Oct 23, 2020
@NotWoods NotWoods merged commit df56bfb into rollup:master Oct 23, 2020
@dummdidumm dummdidumm deleted the patch-1 branch October 24, 2020 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants