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

Versioning and publishing packages #5

Closed
NMinhNguyen opened this issue May 21, 2020 · 8 comments · Fixed by #71
Closed

Versioning and publishing packages #5

NMinhNguyen opened this issue May 21, 2020 · 8 comments · Fixed by #71

Comments

@NMinhNguyen
Copy link
Contributor

NMinhNguyen commented May 21, 2020

Consider adding https://github.com/atlassian/changesets (see #1 (comment))

@sebinsua
Copy link
Contributor

I think we consider this at some point, but not until we are ready to publish packages.

@threepointone
Copy link
Contributor

Since we're now looking at primarily working inside one repo where workspace packages won't be getting published, this isn't as useful as one would image, and as such shouldn't be considered core. Happy to reopen or make a fresh issue later if the actual need comes up.

@sebinsua
Copy link
Contributor

sebinsua commented Aug 19, 2020

I'm going to re-open this since it's noticeably awkward to build and publish right now without any automation.

For example, within the E2E tests, I had to hardcode package names:

// Build and publish packages to the local registry.
for (const packageName of [
'cra-template-modular-typescript',
'create-modular-react-app',
'eslint-config-modular-app',
'modular-scripts',
'modular-template-widget-typescript',
]) {
try {
await execa('yarn', ['workspace', packageName, 'build']);
} catch (error) {
// eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
if (error.stderr && error.stderr.includes('Command "build" not found.')) {
console.debug(
`No "build" script found for ${packageName}. Skipping...`,
);
} else {
throw error;
}
}
await execa(
'yarn',
[
'workspace',
packageName,
'publish',
'--registry',
LOCAL_VERDACCIO_REGISTRY,
'--no-git-tag-version',
'--non-interactive',
'--patch',
],
{ stdio: 'inherit' },
);
}

I'm thinking of installing manypkg and changesets.

@sebinsua sebinsua reopened this Aug 19, 2020
@NMinhNguyen
Copy link
Contributor Author

Since we're now looking at primarily working inside one repo where workspace packages won't be getting published, this isn't as useful as one would image, and as such shouldn't be considered core. Happy to reopen or make a fresh issue later if the actual need comes up.

I think the reason this issue was closed is actually not right anyway. The point of this issue was to automate versioning and publishing of modular-scripts and all of the packages in this repo. It's not about enabling publishing in a monorepo created by create-modular-react-app.

@threepointone
Copy link
Contributor

My bad, sorry!

@threepointone
Copy link
Contributor

(Besides, my original point doesn’t stand either, since we’re now going to publish packages too)

@NMinhNguyen
Copy link
Contributor Author

(Besides, my original point doesn’t stand either, since we’re now going to publish packages too)

Packages in the modular-created monorepo or in standalone repos for consumption by the monorepo?

@threepointone
Copy link
Contributor

modular created repos (per “modular for libs”)

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 a pull request may close this issue.

3 participants