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

Support for tags with prefix (rebased) #76

Closed
wants to merge 3 commits into from

Conversation

tony
Copy link

@tony tony commented Jun 23, 2018

Rebased version of #54 by @scolby33

I'm not fully sure if scolby gets full credit if I rebase and make a PR. @scolby you're free to use this rebase in your own branch/PR if you'd like. I need this feature on several projects, because they all have broken links at the moment.

See also: #65

@tony tony force-pushed the future-releases-uri-config branch from f3a50a8 to 37f53d9 Compare June 23, 2018 19:46
tony added a commit to tmux-python/libtmux that referenced this pull request Jun 23, 2018
See also:
- bitprophet/releases#54 (original PR by
  @scolby33)
- bitprophet/releases#76 (rebased branch of
  above PR used in this commit)
- Feature request: bitprophet/releases#65
@tony
Copy link
Author

tony commented Jun 23, 2018

@scolby33
Copy link

Thanks @tony for the update on this. I have done rebases on pull requests before and I think the commit authorship is saved, unless you squash everything(?). Either way, I'm not too concerned about the credit and am happy that my contributions were useful!

@bitprophet bitprophet added this to the 1.5 milestone Jun 28, 2018
…'t have the same link as already existing releases.

(Git tags vs. a current development branch, for example)
@tony tony force-pushed the future-releases-uri-config branch from 37f53d9 to 9ad1be2 Compare July 8, 2018 20:48
@tony
Copy link
Author

tony commented Jul 8, 2018

@scolby33 @bitprophet Rebased against 8c01876

@tony tony force-pushed the future-releases-uri-config branch from 9ad1be2 to 5ef160a Compare July 21, 2018 15:32
@tony
Copy link
Author

tony commented Jul 21, 2018

@bitprophet How about now?

tony added a commit to tmux-python/tmuxp that referenced this pull request May 12, 2019
Remove releases in favor of a more sphinx-issues and basic reST structuring.
https://github.com/bitprophet/releases was inflexible in cases where we used
v<versionnumber> (v1.2.2) and wanted to link back to master.

Issue / PR:
- bitprophet/releases#54
- bitprophet/releases#76

Other issues is how the behavior did more than styling and issue linking,
sometimes patches only intended for minor releases were also added
to minor releases. Regardless of what it intended to do, it wasn't
compatible with how releases were cut. It does more than the projects
I added to needed.

Instead of using reST sections, it used list items to represent versions. 
Which then go on to build html that a section would anyway.  This feels
out of place when separating versions.

Custom roles used in releases were way too much investment for a
changelog. It'd be worth seeing if the same job could be done by
iterating over node test and linking to issues like GitHub does. e.g #484
@tony
Copy link
Author

tony commented May 26, 2020

Sorry this is stale, someone else is okay to carry on the torch - I don't use this package anymore (just prefer plain files now)

I will close this, but keep my branch up in case anyone wants to continue (in a fresh PR)

@tony tony closed this May 26, 2020
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

Successfully merging this pull request may close these issues.

3 participants