Skip to content

chore(types) remove bespoke rollup types#3009

Merged
rwaskiewicz merged 1 commit intorwaskiewicz-rebase-again-sourcemapsfrom
rwaskiewicz/sourcemaps/remove-custom-rollup-types
Aug 19, 2021
Merged

chore(types) remove bespoke rollup types#3009
rwaskiewicz merged 1 commit intorwaskiewicz-rebase-again-sourcemapsfrom
rwaskiewicz/sourcemaps/remove-custom-rollup-types

Conversation

@rwaskiewicz
Copy link
Copy Markdown
Member

this PR replaces our custom typings for rollup hooks with those
provided by rollup

since this is only a change to the typings, I did not do any functional testing, rather ensured the compiler still ran

this PR replaces our custom typings for rollup hooks with those
provided by rollup
@rwaskiewicz rwaskiewicz requested a review from a team August 18, 2021 20:53
Copy link
Copy Markdown
Contributor

@splitinfinities splitinfinities left a comment

Choose a reason for hiding this comment

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

LGTM!

@rwaskiewicz rwaskiewicz merged commit d4782fb into rwaskiewicz-rebase-again-sourcemaps Aug 19, 2021
@rwaskiewicz rwaskiewicz deleted the rwaskiewicz/sourcemaps/remove-custom-rollup-types branch August 19, 2021 19:09
rwaskiewicz added a commit that referenced this pull request Sep 20, 2021
this PR replaces our custom typings for rollup hooks with those
provided by rollup
rwaskiewicz added a commit that referenced this pull request Sep 30, 2021
duplicate the typings for rollup (again).

Stencil cannot resolve the rollup typings in the private compiler api
typings at run time for consuming projects. As a result, we duplicate
the sourcemap typings from rollup

partial revert of #3009
rwaskiewicz added a commit that referenced this pull request Oct 5, 2021
duplicate the typings for rollup (again).

Stencil cannot resolve the rollup typings in the private compiler api
typings at run time for consuming projects. As a result, we duplicate
the sourcemap typings from rollup

partial revert of #3009
johnjenkins pushed a commit to johnjenkins/stencil that referenced this pull request Oct 7, 2021
this PR replaces our custom typings for rollup hooks with those
provided by rollup
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