Handle "revert" commits in blogposts

eslint / eslint-release

The ESLint release tool

Other

24 stars 11 forks source link

Open ilyavolodin opened 7 years ago

ilyavolodin commented 7 years ago

Currently, we remove "Revert" commits from the blog posts, this unit tests seems to indicated that's a correct behavior: https://github.com/eslint/eslint-release/blob/6e4bd2d43c41904f7428e1828a5501eac26dfb49/tests/lib/release-ops.js#L147

If revert happens a different release then original commit, we should probably show it in blog post.

btmills commented 7 years ago

we should probably show it in blog post

While excluding it from version calculation, yes?

ilyavolodin commented 7 years ago

I think so. I don't think we should ever include "revert" into version calculation.