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

Commit compiled js needed to release v7.10.0 #2327

Merged
merged 1 commit into from
Aug 31, 2020
Merged

Conversation

mejackreed
Copy link
Contributor

No description provided.

@cdmo
Copy link
Member

cdmo commented Aug 28, 2020

Thanks Jack.

Looks like 161 rubocop offenses. And there's this PR that looks like it's intended to fix this problem #2298. That PR needs a rebase though.

Copy link
Member

@cdmo cdmo left a comment

Choose a reason for hiding this comment

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

So, just so I understand the process, this change was produced by running npm run js-compile-bundle locally. Once merged Jack you will run npm publish. Is that right?

I know this page contains details for general releases, but since that's not really what's going on here I just wanted to check my understanding. Also might it be a good idea, separately, to update that doc because it seems pretty different than reality? Like "Create a milestone in GitHub for the NEXT version." for example.

@mejackreed
Copy link
Contributor Author

Yes that is what I am doing.

I think generally most of that is stuff that should happen when creating release. And likely the milestone and email are the things that are being skipped. But I'd say we should strive for it so I would be 👍 for leaving it in.

@cdmo cdmo merged commit c5d587d into master Aug 31, 2020
@cdmo cdmo deleted the commit-compiled-js branch August 31, 2020 16:11
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.

None yet

2 participants