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.

Sign up for GitHub

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

Jump to bottom

Move all build tasks to Rollup #306

Merged
merged 3 commits into from
Jun 18, 2019
Merged

Move all build tasks to Rollup #306

merged 3 commits into from
Jun 18, 2019

Conversation

kitten
Copy link
Member

@kitten kitten commented Jun 16, 2019 •
edited
Loading

  • Declarations are now generated as part of the Rollup build
  • The Rollup build overrides the tsconfig correctly now and no changes to the output were made
  • The clean task now runs only before publishing as that's the only time we care about it
  • Prune was removed; instead we just exclude test files inside Rollup

@kitten kitten requested a review from andyrichardson June 16, 2019 14:36
andyrichardson
andyrichardson reviewed Jun 16, 2019
View reviewed changes
package.json Show resolved Hide resolved
rollup.config.js Show resolved Hide resolved
@kitten kitten requested a review from andyrichardson June 17, 2019 16:56
@andyrichardson andyrichardson merged commit b4f62a9 into master Jun 18, 2019
@andyrichardson andyrichardson deleted the fix/less-npm-tasks branch June 18, 2019 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andyrichardson andyrichardson andyrichardson approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kitten @andyrichardson