changelog icon indicating copy to clipboard operation
changelog copied to clipboard

Feature - control branches on which to generate a change log.

Open newbish opened this issue 2 years ago • 30 comments

Disable building of change log for specified branches while still performing the other release functionality. resolves ​#156, resolves ​#51

newbish avatar Aug 18 '21 05:08 newbish

Requesting a review from @pvdlg or another maintainer.

newbish avatar Aug 24 '21 00:08 newbish

Why not just use array instead of comma separated string on ignoreBranches, while we are using json object as configuration parameter?

kunish avatar Sep 14 '21 04:09 kunish

Why not just use array instead of comma separated string on ignoreBranches, while we are using json object as configuration parameter?

happy to make that change, especially if it gets this feature merged

kkikta-bcg avatar Sep 14 '21 22:09 kkikta-bcg

I am in need of this feature. How would you feel about the option being "branches: [...]" (as opposed to ignoreBranches)? This would have symmetry with the top-level "branches" array in .releaserc. I imagine the repos that would use this would have a list in mind where they do want the changelog modified, and a longer list of branches where they don't.

diff --git a/.releaserc.json b/.releaserc.json
index 094a34a..f5eb394 100644
--- a/.releaserc.json
+++ b/.releaserc.json
@@ -6,6 +6,7 @@
       "@semantic-release/changelog",
       {
         "changelogFile": "CHANGELOG.md",
+        "branches": ["master"]
       }
     ],

kingphil avatar Sep 24 '21 17:09 kingphil

requesting review by a maintainer, @gr2m perhaps?

newbish avatar Sep 29 '21 06:09 newbish

resolves #156

newbish avatar Oct 08 '21 05:10 newbish

resolves #51

newbish avatar Oct 08 '21 05:10 newbish

+1 We would love this to be available asap

eLarocque avatar Oct 25 '21 14:10 eLarocque

+1 Looking for this too. I can put in a workaround mentioned in #51, but would love to see this supported OOTB. Nice work @newbish

cgreene-st avatar Oct 29 '21 03:10 cgreene-st

Why did you close it? Is it done or don't you want to wait for approvals anymore? 🙈

Roseidon avatar Dec 04 '21 18:12 Roseidon

Why did you close it? Is it done or don't you want to wait for approvals anymore? 🙈

Realized it wasn't working the way I wanted it to. So did some rewrite since I needed it working and wanted to get a version in NPM ;)

https://www.npmjs.com/package/@newbish/changelog

newbish avatar Dec 05 '21 08:12 newbish

Feel free to test it out and see if it meets your needs, I was using this repo to test: https://github.com/newbish/changelog-test

newbish avatar Dec 05 '21 08:12 newbish

Hi everyone! Any reviewer to take a look into this?

@newbish could you resolve the conflicts?

leandrocrs avatar Feb 02 '22 14:02 leandrocrs

resolved conflicts

newbish avatar Feb 10 '22 17:02 newbish

@alexsanderp do you mind taking another look?

newbish avatar Feb 10 '22 17:02 newbish

I feel like having the option to filter all plugins by branches:[...] would be a good thing.

Is this also possible with this PR or does it only work with the changelog plugin?

Silthus avatar Feb 19 '22 07:02 Silthus

I feel like having the option to filter all plugins by branches:[...] would be a good thing.

Is this also possible with this PR or does it only work with the changelog plugin?

This only affects the changelog, I believe the other plugins would have to implement something similar to have that capability.

newbish avatar Feb 19 '22 14:02 newbish

This only affects the changelog, I believe the other plugins would have to implement something similar to have that capability.

Do you think there is some way to implement this for all plugins at once in the core library?

Silthus avatar Feb 19 '22 15:02 Silthus

@alexsanderp can you please re-review this PR

newbish avatar Feb 25 '22 05:02 newbish

@gr2m would you be able to provide a review?

newbish avatar Mar 11 '22 05:03 newbish

I believe some changes to files formatting could be out by this time.

attempted to apply prettier to match current formatting.

newbish avatar Mar 23 '22 01:03 newbish

@newbish awesome PR 🎉 @alexsanderp, @gr2m it is possible that we merge this PR, it would be really awesome 🙏🏼

mario-jerkovic avatar Apr 21 '22 13:04 mario-jerkovic

@travi would you by chance be able to provide a review?

newbish avatar Apr 23 '22 23:04 newbish

Anyone from the semantic-release team? @boennemann @UziTech @keplersj @kbrandwijk

newbish avatar Apr 27 '22 15:04 newbish

LGTM but I don't have write permission for this repo.

UziTech avatar Apr 28 '22 03:04 UziTech

@alexsanderp, thanks for the approval, however, looks like it requires maintainer approval.

newbish avatar May 18 '22 00:05 newbish

Hey, is it going to be merged?

mgolawski avatar May 20 '22 11:05 mgolawski

LGTM but I don't have write permission for this repo.

any chance you can tell who has maintainer?

newbish avatar May 24 '22 03:05 newbish

LGTM but I don't have write permission for this repo.

any chance you can tell who has maintainer?

When looking at previously closed PRs, you can see one was closed and approved by @travi in 2021

Dunno of he's still a maintainer but might be worth a shot

eLarocque avatar May 24 '22 05:05 eLarocque

i'm sorry that there hasn't been much maintainer activity on this thread, but our maintenance team has been very busy and unable to keep up with all of the activity across the projects.

i think it is important to point to a few key points from our contributing guide:

  • https://github.com/semantic-release/semantic-release/blob/ab45ab1f8d0d0f728fb026a92ec191bcc88f51f1/CONTRIBUTING.md?plain=1#L55
  • https://github.com/semantic-release/semantic-release/blob/ab45ab1f8d0d0f728fb026a92ec191bcc88f51f1/CONTRIBUTING.md?plain=1#L68

in the various threads that are linked from this PR, i do not find any mention of this feature request being approved by a maintainer. i also do not find any responses to maintainer questions for clarification around why the feature is needed for supported workflows. instead the described desire details workflows that are not officially supported uses of the pre-release workflow functionality. pre-releases are intended for situations like confirming a breaking api change or rare changes that cannot be tested in isolation because of built tool changes or similar. pre-releases are not intended to be used to enable workflows similar to gitflow where every change starts as a pre-release to enable verification through the package consumers before promoting to the stable release line. keep in mind that semantic-release is not a generic workflow tool, but an opinionated tool that is intended to encourage continuous deployment style practices and simplify semantic versioning and the communication of the impact of those changes. even if we were targeting a more generic workflow tool, decisions have to be made to keep the project maintainable.

as mentioned, our maintenance team has not been able to keep up with all activity, so we have to make decisions around priority. unsupported usage naturally gets prioritized lower than supported usage. without the information requested in our contributing guide, we are left to assume that the request is to enable unsupported workflows like i describe above. we have not closed this PR because we are not deciding yet that this will never be implemented, but it is being treated as lower priority based on the information we currently have. if the assumptions are being made with incomplete/incorrect information, the next step should be to provide a more complete description of the desired use case and why it falls within the supported workflows. in order for a change like this one to move forward, the maintainers need to agree that it is valuable to the project and is worth the additional complexity and maintenance burden moving forward.

travi avatar May 24 '22 05:05 travi