Skip to content

ENH: Allow to manually define which pyproject.toml file to use - #156

Closed
seberg wants to merge 1 commit into
twisted:masterfrom
seberg:optional-pyproject
Closed

seberg wants to merge 1 commit into
twisted:masterfrom
seberg:optional-pyproject

Conversation

@seberg

@seberg seberg commented Jul 17, 2019

Copy link
Copy Markdown

This adds --pyproject to towncrier, the same way as what exists
for towncrier.check.


This probably needs a few additional things (such as tests)? I tried it locally briefly, and it seems to work (is pretty straight forward after all).

@seberg
seberg force-pushed the optional-pyproject branch from c8abebe to 9354e2d Compare July 17, 2019 21:09
@codecov-io

codecov-io commented Jul 17, 2019 •

Copy link
Copy Markdown

Codecov Report

Merging #156 into master will decrease coverage by 0.28%.
The diff coverage is 66.66%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #156      +/-   ##
========================================
- Coverage   93.29%    93%   -0.29%     
========================================
  Files          15     15              
  Lines         626    629       +3     
  Branches       65     66       +1     
========================================
+ Hits          584    585       +1     
- Misses         26     27       +1     
- Partials       16     17       +1
Impacted Files Coverage Δ
src/towncrier/__init__.py 88.23% <66.66%> (-2.54%) ⬇️

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 ecd438c...5d1a830. Read the comment docs.

This adds `--pyproject` to `towncrier`, the same way as what exists
for `towncrier.check`.

Closes twistedgh-148
@seberg
seberg force-pushed the optional-pyproject branch from 9354e2d to 5d1a830 Compare July 18, 2019 02:08
@seberg

seberg commented Jul 18, 2019 •

Copy link
Copy Markdown
Author

xref gh-157 which solves the same issue (uses --config though). EDIT: I do not care which solution is preferably, I chose --pyproject to make it similar to the code in towncrier.check.

@hawkowl

hawkowl commented Aug 11, 2019

Copy link
Copy Markdown
Member

I merged 157, I prefer the config name over pyproject (since pyproject is python specific).

Thanks!

@hawkowl hawkowl closed this Aug 11, 2019
@seberg

seberg commented Aug 11, 2019

Copy link
Copy Markdown
Author

I agree config is nicer, could change towncrier.check to also accept --config, but low priority.

@seberg
seberg deleted the optional-pyproject branch August 11, 2019 21:53
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.

4 participants