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

Add include-labels option #500

Merged
merged 5 commits into from May 26, 2020

Conversation

kris-reynolds
Copy link
Contributor

@kris-reynolds kris-reynolds commented Apr 22, 2020

Adding feature discussed in: #499

Note, feature has not been agreed upon yet. Consider this WIP until feature has consensus.

fixes #499

@kris-reynolds kris-reynolds changed the title Add include-labels option WIP: Add include-labels option Apr 22, 2020
@kris-reynolds kris-reynolds changed the title WIP: Add include-labels option Add include-labels option Apr 22, 2020
@kris-reynolds kris-reynolds changed the title Add include-labels option WIP: Add include-labels option Apr 22, 2020
lib/releases.js Outdated Show resolved Hide resolved
Co-Authored-By: Joseph Petersen <josephp90@gmail.com>
@jetersen jetersen marked this pull request as draft April 23, 2020 08:13
@kris-reynolds kris-reynolds changed the title WIP: Add include-labels option Add include-labels option Apr 27, 2020
@kris-reynolds kris-reynolds marked this pull request as ready for review April 28, 2020 13:45
Copy link
Member

@jetersen jetersen left a comment

Choose a reason for hiding this comment

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

LGTM

@jetersen
Copy link
Member

Is this still to be considered WIP @kreyusb ?

@kris-reynolds
Copy link
Contributor Author

@jetersen We have been using it like this on our Company configuration for the last couple of weeks with no changes. I would say it is good to go.

@darkdreamingdan
Copy link
Contributor

Can we go ahead and merge this? Seems like everyone is happy. I see nothing blocking and concerned good code is just getting stuck and we're getting merge conflicts.

@jetersen
Copy link
Member

jetersen commented May 26, 2020

There is currently a conflict, do you mind solving it kreyusb?
Than we are ready to merge.

@jetersen jetersen added the type: feature New feature or request label May 26, 2020
@jetersen
Copy link
Member

jetersen commented May 26, 2020

Many thanks for your contribution! 👏

@jetersen jetersen merged commit f8a15f0 into release-drafter:master May 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add config option for include-labels
3 participants