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

changelog: categorize changelog entry using labels #4474

Merged
merged 1 commit into from
Aug 26, 2020

Conversation

skshetry
Copy link
Member

With this change, release-drafter will be able to categorically
write/create changelogs based on the labels.

Check this release which was generated by same config: https://github.com/skshetry/dvc/releases/tag/1.2.4

Fixes #4023

Thank you for the contribution - we'll try to review it as soon as possible. πŸ™

With this change, release-drafter will be able to categorically
write/create changelogs based on the labels.
@skshetry skshetry added this to In progress in DVC 25 August - 8 September 2020 via automation Aug 26, 2020
@skshetry skshetry self-assigned this Aug 26, 2020
categories:
- title: "πŸš€ New Features and Enhancements "
labels:
- "feature"
Copy link
Member Author

Choose a reason for hiding this comment

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

Could also start using labels with emojis.

@@ -1,5 +1,48 @@
# Config for https://github.com/apps/release-drafter
branches:
- master
name-template: "$RESOLVED_VERSION πŸ¦‰ "
Copy link
Member Author

Choose a reason for hiding this comment

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

Previously, it'd generate draft releases with "Draft" title. Now, it can automatically resolve versions. To auto-increment, slap a "minor"/"major"/"patch" labels on PR and it'll auto-increment after merge.

Copy link
Contributor

Choose a reason for hiding this comment

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

Whoa...

Copy link
Contributor

@casperdcl casperdcl Aug 26, 2020

Choose a reason for hiding this comment

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

Suggested change
name-template: "$RESOLVED_VERSION πŸ¦‰ "
name-template: "$RESOLVED_VERSION πŸ¦‰"

(minor whitespace :P)

I also noticed a lot of whitespaces for other titles - is this to support potential suffixes like "minor/major?" Surely the suffix should add the space?

Copy link
Member Author

Choose a reason for hiding this comment

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

Mostly a copy-paste issue with one that I was playing with on my private repo. πŸ˜„ We can leave it for now.

@skshetry skshetry moved this from In progress to Review in progress in DVC 25 August - 8 September 2020 Aug 26, 2020
@skshetry skshetry added the chore label Aug 26, 2020
Copy link
Contributor

@pared pared left a comment

Choose a reason for hiding this comment

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

So we need to introduce few new labels after this change is merged?

@skshetry
Copy link
Member Author

skshetry commented Aug 26, 2020

@pared, yes, we do need some labels. Let me track it here.

  • feature
  • enhancement
  • breaking-change
  • optimize
  • performance
  • refactoring
  • bugfix
  • bug
  • maintenance
  • chore
  • build
  • ci
  • skip-changelog
  • major
  • minor
  • patch

@skshetry
Copy link
Member Author

Okay, moving ahead with this. We can always change the labels or the "texts".

@skshetry skshetry merged commit 54deca7 into iterative:master Aug 26, 2020
DVC 25 August - 8 September 2020 automation moved this from Review in progress to Done Aug 26, 2020
@skshetry skshetry deleted the changelog branch August 26, 2020 15:50
Comment on lines +47 to +48

Thanks again to $CONTRIBUTORS for the contributions! πŸŽ‰
Copy link
Contributor

Choose a reason for hiding this comment

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

weird to @mention everyone again a second time. waste of a line?

Copy link
Member Author

Choose a reason for hiding this comment

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

Nope, I don't think it's enough to thank people for their contributions. Maybe add another line too? 😜

Copy link
Member Author

@skshetry skshetry Aug 26, 2020

Choose a reason for hiding this comment

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

Anyway, they don't get notifications, so should be fine, I guess.

Copy link
Contributor

Choose a reason for hiding this comment

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

to be clear, each issue/PR changelog entry already has the person @mentioned, which is already more than most changelogs do and is also arguably besides the point of a changelog. Having another line at the bottom mentioning everyone again is pointless. At that stage, yes, you may as well really add another line soliciting additional contributions and linking to https://dvc.org/doc/user-guide/contributing/core

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

keep a human-readable changelog
4 participants