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

Prepare CHANGELOG and release planning #2299

Merged
merged 2 commits into from
Jan 31, 2017
Merged

Prepare CHANGELOG and release planning #2299

merged 2 commits into from
Jan 31, 2017

Conversation

jeddy3
Copy link
Member

@jeddy3 jeddy3 commented Jan 29, 2017

Tried to remove the redundancy from the changelog to make it clearer and easier to read. Updated the release planning doc to only include things that requires the community’s help e.g. new plugins or helping out with existing ones (e.g. stylelint-order)

Copy link
Member

@hudochenkov hudochenkov left a comment

Choose a reason for hiding this comment

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

Much cleaner! Thank you!

CHANGELOG.md Outdated
- Deprecated: `"at-rules-without-declaration-blocks"` option for `max-nesting-depth`. Use the `"blockless-at-rules"` option instead.
See the [release planning](docs/user-guide/release-planning.md) document for information on some of these rule deprecations.

- Deprecated: 13 rules ([#2197](https://github.com/stylelint/stylelint/pull/2197) & [#2285](https://github.com/stylelint/stylelint/pull/2285)).
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should be more verbose here, because these are major changes and users will want to know what's going on. How about grouping rules by category and providing suggestions about what to do (what to use & contribute to) if you liked the rule(s). For example, all of the spec validation rules could be grouped with a statement like this:

It is beyond the scope of stylelint's core package to effectively validate against the CSS spec. Please investigate csstree and css-values for this functionality, and contribute to those projects. csstree already has a stylelint plugin, and css-values needs one to be developed.

Copy link
Member Author

Choose a reason for hiding this comment

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

because these are major changes and users will want to know what's going on

Agreed, but I thought that is what the release planning document was for? i.e. a canonical place we can keep up to date with evolving suggestions about what to do leading up a release.

For example, all of the spec validation rules could be grouped with a statement like this:

Agreed. At the moment I've split out the wrapping rules into their own group and kept declaration-block-properties-order by itself. Splitting out the validation rules into their own group makes sense. Any other suggestions?

Copy link
Contributor

Choose a reason for hiding this comment

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

I thought that is what the release planning document was for

I think many, many fewer people will read the release planning document than the changelog. I, for example, have never bothered to read a release planning document ever. Changelog is the place you look when changes have been made, and you want to know what they are, how to adapt — a release planning document sounds like the place you look when you are trying to contribute or wondering what is coming next.

Splitting out the validation rules into their own group makes sense. Any other suggestions?

Yeah, I think every rule should have an explanation about what to do next. So:

  • Rules that simply wrap a third-party library. We encourage users to create and help maintain plugins for these rules.
    • no-browser-hacks
    • no-indistinguishable-colors
    • no-unsupported-browser-features
  • Rules that did not seem useful. If you liked these rules, please create plugins for them.
    • root-no-standard-properties
    • custom-property-no-outside-root
  • stylelint-disable-reason did not work well. Please consider contributing to Add reportUnscopedDisables flag #2292 for a replacement.

I think that the above, combined with spec validation and rules that already have a note, should cover it.

Copy link
Member Author

Choose a reason for hiding this comment

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

I think many, many fewer people will read the release planning document than the changelog. I, for example, have never bothered to read a release planning document ever. Changelog is the place you look when changes have been made, and you want to know what they are, how to adapt

I agree. I'm trying to remember why we did the releasing planning document split for 7.0.0, but it escapes me. Anyhow, I'm agreement. I'll update CHANGELOG and retire the releasing planning doc.

Copy link
Member

Choose a reason for hiding this comment

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

❤️

@jeddy3
Copy link
Member Author

jeddy3 commented Jan 29, 2017

Sorry guys, it got late here. I've pushed up what I've got so far. I can pick things up tomorrow if no else gets around to it before then.

@jeddy3 jeddy3 force-pushed the cl-and-rp branch 2 times, most recently from d870e3a to 951592d Compare January 30, 2017 22:54
@jeddy3
Copy link
Member Author

jeddy3 commented Jan 30, 2017

It took a little longer than expected (as there were a few things to iron out across the repos), but things are looking good for a release now.

The demo, standard config and stylelint.io repos now all look good when used with this branch (which is rebased off master). Tests are passing and the links on the website appear dandy.

@davidtheclark I'm going to have to bail for the night. Are you able to update the CHANGELOG in this PR/branch to your liking? I can then give it a quick once over in the morning before releasing. Sound good?

@davidtheclark
Copy link
Contributor

@jeddy3 I just committed modifications to the changelog. Thanks for preparing this release!

@jeddy3
Copy link
Member Author

jeddy3 commented Jan 31, 2017

I just committed modifications to the changelog.

Awesome. Looks great!

@jeddy3 jeddy3 merged commit e484170 into master Jan 31, 2017
@jeddy3 jeddy3 deleted the cl-and-rp branch January 31, 2017 15:00
@ntwb
Copy link
Member

ntwb commented Jan 31, 2017

"Updated the release planning doc to only include things that requires the community’s help "

@jeddy3 Rather than "Updated" the entire "release planning doc" markdown file and link from the user guide have been removed, I just wanted to ask to confirm that this was intentional?

@hudochenkov
Copy link
Member

@ntwb it was intentional. Look at thread #2299 (comment)

@ntwb
Copy link
Member

ntwb commented Jan 31, 2017

@hudochenkov Awesome, thanks, I didn't expand the diff to see those comments.

I think this is a fantastic change, my goto is to always to read the release notes, if there are none then I'll start scouring a repo for a changelog.md or changelog in the readme.md but as David pointed out never a release planning doc 💯

sergesemashko pushed a commit to sergesemashko/stylelint that referenced this pull request Mar 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants