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

Release v1.2.0 #1765

Merged
merged 1 commit into from Sep 28, 2020
Merged

Release v1.2.0 #1765

merged 1 commit into from Sep 28, 2020

Conversation

UziTech
Copy link
Member

@UziTech UziTech commented Sep 27, 2020

Release Notes


Features

Fixes

Dependencies

Documentation


Publisher

  • $ npm version has been run.
  • Release notes in draft GitHub release are up to date
  • Release notes include which flavors and versions of Markdown are supported by this release
  • Committer checklist is complete.
  • Merge PR.
  • Publish GitHub release using master with correct version number.
  • $ npm publish has been run.
  • Create draft GitHub release to prepare next release.

Note: If merges to master occur after submitting this PR and before running $ npm pubish you should be able to

  1. pull from upstream/master (git pull upstream master) into the branch holding this version,
  2. run $ npm run build to regenerate the min file, and
  3. commit and push the updated changes.

Committer

In most cases, this should be someone different than the publisher.

  • Version in package.json has been updated (see PUBLISHING.md).
  • The marked.min.js has been updated; or,
  • release does not change library.
  • CI is green (no forced merge required).

@vercel
Copy link

vercel bot commented Sep 27, 2020

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

🔍 Inspect: https://vercel.com/markedjs/markedjs/pzhqpxs4x
✅ Preview: https://markedjs-git-fork-uzitech-release-120.markedjs.vercel.app

@joshbruce
Copy link
Member

@UziTech and @styfle: Before I push the release, want to make sure the checklist instructions are still correct - been a minute for me.

@UziTech
Copy link
Member Author

UziTech commented Sep 28, 2020

Ya just need to make sure you pull the changes in master after this PR is merged, then run npm publish. If you want me to do it I can.

@joshbruce
Copy link
Member

Would you please? Have to mess it up from lack of practice. (Been in PHP deployment land.)

@UziTech UziTech merged commit cdaf80a into markedjs:master Sep 28, 2020
@UziTech UziTech deleted the release-1.2.0 branch September 28, 2020 01:04
@styfle
Copy link
Member

styfle commented Sep 28, 2020

We could probably remove the checkboxes for the following:

  • The marked.min.js has been updated; or,
  • release does not change library.

Since we have the bot, the minified file should always be up to date.

@UziTech
Copy link
Member Author

UziTech commented Sep 28, 2020

What do you guys think about moving to semantic release. I think we talked about it before and decided to wait until after v1.0.

@styfle
Copy link
Member

styfle commented Sep 28, 2020

I haven't used it before but it sounds good to me 👍

Would it be better to change to squash merge at the same time so we don't need to waste time with commit messages in PRs, just the merged commit message?

@UziTech
Copy link
Member Author

UziTech commented Sep 28, 2020

ya or we could use commitlint to enforce commit messages. I tend to prefer squashing though.

@styfle
Copy link
Member

styfle commented Sep 28, 2020

The reason why I dislike commit lint is that it discourages outside collaboration, especially new users who don't understand git amend or git rebase.

Lets do squash merging + semantic release 🚀

@joshbruce
Copy link
Member

Thought we already were - soon as we hit 1.0 the expectation would be minor (new feature, non-BC) and patch (bug fixes). ??

Think some of the roadmap notes are still available in the projects tab.

I'm a good example of what @styfle is talking about. I don't use Git CLI because I typically don't have a need for features beyond what I get from GitHub Desktop. I've debated about switching to Squash to reduce number of commits recorded (if I read up on it properly) but still haven't.

@styfle
Copy link
Member

styfle commented Sep 28, 2020

Great! I switched to squash merging for PRs going forward 👍

@UziTech Do you want to enable semantic release since you're more familiar with it?

@UziTech
Copy link
Member Author

UziTech commented Sep 28, 2020

Ya I'll create a PR shortly to use semantic release.

@github-actions
Copy link

🎉 This PR is included in version 1.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants