-
Notifications
You must be signed in to change notification settings - Fork 2
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
chore: prepare for first stable release #822
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
- Could you add an issue to update the readme? (Or just update it if that's faster.)
- I'll clean up Authors right now.
- Could you make sure
CHANGELOG.md
is ready to go? I think we probably just want to delete it entirely.
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "@mapeo/core", | |||
"version": "9.0.0-alpha.23", | |||
"name": "@comapeo/core", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, could you run npm install
before merging? I think that will update package-lock.json
.
This needs first the publishing of mapeo-schema v1 before merging |
should close #752
Additionally we are:
@comapeo/core
We could also:
Early development
disclaimer and correctnpm
command)This PR is also made so that we can test OPTICS for releasing a stable version.
We should have an NPM_TOKEN on the project before merging the OPTICS_RELEASE so we can actually publish the package.
On the other hand, I'm wondering if PRs merged after this, would be included in the release (I think yes?), but if not, we should make sure we're not missing anything (I think not)