[GitHub] [superset] villebro commented on issue #13013: [SIP-58] Proposal to move superset-ui to this repo

[GitBox <gi...@apache.org>]

villebro commented on issue #13013:
URL: https://github.com/apache/superset/issues/13013#issuecomment-958178518


   Hi all,
   
   after carefully considering all points brought up in this discussion, we have done some prerequisite work required by the monorepo effort to more closely align the `superset-ui` repo with the main Superset repo. Some notable PRs:
   - Add license headers to all files on `superset-ui`: https://github.com/apache-superset/superset-ui/pull/1444
   - Migrate `superset-ui` from `yarn` to `npm` as per [SIP-14](https://github.com/apache/superset/issues/6217): https://github.com/apache-superset/superset-ui/pull/1405
   - Upgrade `superset-ui` Storybook to version 6 to match the version used on `apache/superset`: https://github.com/apache-superset/superset-ui/pull/1409
   
   Based on initial testing, both repos should now be in a state that should make the monorepo transition possible. In the following doc we summarize some of the key motivations for migrating the `superset-ui` code to the main Superset repo, consider pros/cons of the various alternatives proposed in this SIP and present a proposed solution for performing the migration:
   
   [Spike_-_Superset_Monorepo.pdf](https://github.com/apache/superset/files/7463988/Spike_-_Superset_Monorepo.pdf)
   
   A vote for the SIP will be initiated shortly.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org