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

Allow for multiple fragments with the same ticket and type #119

Merged
merged 9 commits into from
Sep 6, 2018

Conversation

Cadair
Copy link
Contributor

@Cadair Cadair commented May 30, 2018

This extends the fragment format to be <issue>.<type>(.<counter>) i.e have an optional .counter after the type, which needs to be an integer.

This extends the fragment format to be <issue>.<type>(.<counter>) i.e have an
optional .counter after the type.
@codecov-io
Copy link

codecov-io commented May 30, 2018

Codecov Report

Merging #119 into master will increase coverage by 0.06%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #119      +/-   ##
==========================================
+ Coverage   93.37%   93.43%   +0.06%     
==========================================
  Files          15       15              
  Lines         634      640       +6     
  Branches       64       65       +1     
==========================================
+ Hits          592      598       +6     
  Misses         26       26              
  Partials       16       16
Impacted Files Coverage Δ
src/towncrier/test/test_write.py 100% <ø> (ø) ⬆️
src/towncrier/test/test_format.py 100% <ø> (ø) ⬆️
src/towncrier/_builder.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d31ccdc...009c5b4. Read the comment docs.

Cadair added a commit to Cadair/dkist that referenced this pull request May 30, 2018
@Cadair Cadair force-pushed the multiple_fragments branch from de07382 to c64199e Compare May 30, 2018 15:00
@Cadair Cadair force-pushed the multiple_fragments branch from c64199e to 7196fab Compare May 30, 2018 15:07
@hawkowl
Copy link
Member

hawkowl commented Jun 12, 2018

@Cadair Could you write a changelog entry for this? :)

@Cadair
Copy link
Contributor Author

Cadair commented Jun 12, 2018

done :)

turt2live added a commit to turt2live/matrix-doc that referenced this pull request Jul 12, 2018
Specifying multiple entries for the same PR would be covered under: twisted/towncrier#119
@Cadair
Copy link
Contributor Author

Cadair commented Jul 19, 2018

ping? 👼

Anything else that needs to happen on this?

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