Skip to content

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

Merged
hawkowl merged 9 commits intotwisted:masterfrom
Cadair:multiple_fragments
Sep 6, 2018
Merged

Allow for multiple fragments with the same ticket and type#119
hawkowl merged 9 commits intotwisted:masterfrom
Cadair:multiple_fragments

Conversation

@Cadair
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Member

hawkowl commented Jun 12, 2018

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

@Cadair
Copy link
Copy Markdown
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
Copy Markdown
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