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

Non-modular templates #2368

Merged
merged 5 commits into from
May 3, 2023
Merged

Conversation

cristiano-belloni
Copy link
Contributor

@cristiano-belloni cristiano-belloni commented May 2, 2023

Not specifying modularTemplateType in the template creates a non-modular target package (i.e. a package that has no modular field in its package.json)

@changeset-bot
Copy link

changeset-bot bot commented May 2, 2023

🦋 Changeset detected

Latest commit: 58a1862

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
modular-scripts Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coveralls
Copy link
Collaborator

coveralls commented May 2, 2023

Coverage Status

Coverage: 49.688% (+0.1%) from 49.552% when pulling 58a1862 on feature/non-modular-templates into d4a64ef on feature/v4.4.

@cristiano-belloni cristiano-belloni marked this pull request as ready for review May 3, 2023 12:24
@cristiano-belloni cristiano-belloni changed the title Non modular templates Non-modular templates May 3, 2023
@@ -0,0 +1,14 @@
{
"name": "modular-template-non-modular",
Copy link
Contributor

Choose a reason for hiding this comment

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

I find this name a little confusing

@cristiano-belloni cristiano-belloni merged commit 2ca7fcd into feature/v4.4 May 3, 2023
@cristiano-belloni cristiano-belloni deleted the feature/non-modular-templates branch May 3, 2023 13:13
@github-actions github-actions bot mentioned this pull request May 9, 2023
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.

4 participants