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(core): a core bundle #138

Merged
merged 6 commits into from
Oct 24, 2019
Merged

feat(core): a core bundle #138

merged 6 commits into from
Oct 24, 2019

Conversation

TheSisb
Copy link
Contributor

@TheSisb TheSisb commented Oct 15, 2019

This PR adds a "Core bundle". This was added to make it easier to start using Paste by combining all of our packages into a single import.

The general approach I took to making this work was by using Lerna to fetch all the packages in our monorepo. I then filter some out and proceed to generate the package.json and index.tsx files.

Open questions:

  • Should any non-core packages be included? (i.e.: theme?)
  • Is where I put the paste-core folder (/packages/paste-core/core-bundle) appropriate?

@TheSisb TheSisb added Type: Enhancement Changes that don't affect the meaning of the code Status: Pls CR This PR is ready for Code Reviews Area: Infrastructure Related to our infrastructure and build tools labels Oct 15, 2019
@TheSisb TheSisb self-assigned this Oct 15, 2019
@vercel
Copy link

vercel bot commented Oct 15, 2019

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/twilio-dsys/paste/7c6ka3sb2
🌍 Preview: https://paste-git-core-bundle.twilio-dsys.now.sh

}

// Main
(async () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Reading this function explains the main procedure that happens right before a build is triggered on this package.

Copy link
Contributor

Choose a reason for hiding this comment

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

Using private, version, and blacklisted seems like a good way to handle that filter. I like it.

@richbachman
Copy link
Contributor

The /packages/paste-core/core-bundle placement makes sense to me.

@vercel vercel bot temporarily deployed to staging October 24, 2019 15:59 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Infrastructure Related to our infrastructure and build tools Status: Pls CR This PR is ready for Code Reviews Type: Enhancement Changes that don't affect the meaning of the code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants