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

feat(build): Add custom rollup config for core #9297

Merged
merged 1 commit into from
Jun 5, 2021

Conversation

vigneshm
Copy link
Contributor

@vigneshm vigneshm commented Jun 4, 2021

Depends on #9295

The core package has some unique characteristics that need a custom rollup config file rather than using the generic one from @spinnaker/scripts. This includes

  • copying over typescript definition files
  • defining an alias mapping for root
  • a plugin to fix typescript's path auto-rewrites.

@vigneshm vigneshm force-pushed the core-rollup-config branch from f0ac239 to 0a9984e Compare June 4, 2021 23:04
Copy link
Contributor

@christopherthielen christopherthielen left a comment

Choose a reason for hiding this comment

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

couple questions but otherwise lgtm

@vigneshm vigneshm force-pushed the core-rollup-config branch from 0a9984e to 3f331a7 Compare June 4, 2021 23:52
The `core` package has some unique characteristics that need a custom rollup config file rather
than using the generic one from `@spinnaker/scripts`. This includes

- copying over typescript definition files
- defining an alias mapping for `root`
- a plugin to fix typescript's path auto-rewrites.
@vigneshm vigneshm force-pushed the core-rollup-config branch from 3f331a7 to 8a06319 Compare June 4, 2021 23:55
@vigneshm vigneshm merged commit e5a953b into spinnaker:master Jun 5, 2021
@vigneshm vigneshm deleted the core-rollup-config branch June 5, 2021 00:10
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.

3 participants