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

Changes from pushes to master (without a PR) #552

Open
ianwalter opened this issue Jun 2, 2020 · 14 comments
Open

Changes from pushes to master (without a PR) #552

ianwalter opened this issue Jun 2, 2020 · 14 comments

Comments

@ianwalter
Copy link
Contributor

I couldn't find an issue related to this so sorry if this is redundant but can release drafter add changes for commits to master not added through a pull request? It doesn't seem like there is a way to do this. If so, are there any workarounds for this?

@jetersen
Copy link
Member

jetersen commented Jun 2, 2020

We have the data to do it as we are pulling commits and associated pull requests, but since it falls outside the categorization. The data is not used.

@ianwalter
Copy link
Contributor Author

ianwalter commented Jun 2, 2020

Thank you that is very helpful to understand. Maybe there should be a default "Other Changes" category or that data can be added to a $OTHER_CHANGES variable? I did see someone asking about integration with Conventional Commits which I think could be related to this as well.

@adrianthedev
Copy link

I see this as a valid use case too.

@albertpastrana
Copy link

We'd love release drafter but we do trunk based development (i.e. no prs) and can't use it because of that.

Do you think this is something that will get some attention @jetersen ?

@jetersen
Copy link
Member

jetersen commented Aug 6, 2020

Only if those doing trunk based development want to contribute 😓

I myself do not use trunk based development so I am not qualified deem what is needed.

@albertpastrana
Copy link

albertpastrana commented Aug 6, 2020 via email

@matfax
Copy link

matfax commented Aug 15, 2020

I would suggest matching semantic commit messages for its underlying categories.
There's the Conventional Commits convention 1.0.0, for example.

fix => Fix
feat => Feature
docs => Documentation
! prefix => minor/major feature

Besides the default set of type tags, the user could also be let to configure their own tags with linked categories.

@lucagiove
Copy link

Another situation that is not working is having PR merged in a branch than merged to master.
Esample
pr-1 -> release-branch
pr-2 -> release-branch
pr-release-branch -> master

Changelog considers only pr-release-branch

@ghost
Copy link

ghost commented Feb 11, 2022

I would like also to add my usecase to this discussion.

I have a module that I publish to pypi, but its small and its only me that works on it. I do use PRs for most features and I have workflows setup to run unit tests for every PR, but sometimes the changes are very small (updating a configuration file or documentation for example) that I do not want to have the action runners triggered for, so I just push to master directly with a commit.

Sometimes, these changes dont matter much, but other times it would be nice to include them as part of the release drafter, like for example I've updated setuptools to include dependencies now (which means im changing my docs to not including installing dependencies separately to my module)

This is absolutely a small concern and an edge case. But thought I would put in my usecase as it definitely would be handy to have some way to say "include this commit to master in the release draft".

@jetersen
Copy link
Member

jetersen commented Mar 7, 2022

I do not see the feature being supported as there is no way to label commits unless your using conventional commits or some other way of categorizing commits.

If anyone is willing to contribute conventional commits support it might be worthwhile.

But I guess not everyone here is on the conventional commits train when doing trunk based development.

@jetersen
Copy link
Member

jetersen commented Mar 7, 2022

Another situation that is not working is having PR merged in a branch than merged to master. Esample pr-1 -> release-branch pr-2 -> release-branch pr-release-branch -> master

Changelog considers only pr-release-branch

No longer true. Release-drafter will now attempt to draft according to the target reference.
Thanks to #1065

@jetersen
Copy link
Member

jetersen commented Mar 7, 2022

Thank you that is very helpful to understand. Maybe there should be a default "Other Changes" category or that data can be added to a $OTHER_CHANGES variable? I did see someone asking about integration with Conventional Commits which I think could be related to this as well.

Other changes was fixed in #1013

@palanga
Copy link

palanga commented Nov 16, 2022

Can't we just put those "uncategorized" commits on an "uncategorized" category instead of discard them ?

@mehdihadeli
Copy link
Contributor

google-github-actions/release-please-action do this with conventional commits and categorize them based on the conventional commits type.

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

No branches or pull requests

8 participants