Skip to content

Fix documentation of fragment configuration in README#425

Merged
adiroiban merged 2 commits intotwisted:trunkfrom
jenshnielsen:fix_readme_typo
Sep 6, 2022
Merged

Fix documentation of fragment configuration in README#425
adiroiban merged 2 commits intotwisted:trunkfrom
jenshnielsen:fix_readme_typo

Conversation

@jenshnielsen
Copy link
Copy Markdown
Contributor

@jenshnielsen jenshnielsen commented Sep 6, 2022

Description

Update to match what the code expects and how this is documented
in https://towncrier.readthedocs.io/en/stable/configuration.html

Fixes #424

Checklist

  • Make sure changes are covered by existing or new tests. N/A Readme only change
  • For at least one Python version, make sure local test run is green. N/A readme only change
  • Create a file in src/towncrier/newsfragments/. Describe your
    change and include important information. Your change will be included in the public release notes.
  • Make sure all GitHub Actions checks are green (they are automatically checking all of the above). N/A Readme only change

@jenshnielsen jenshnielsen requested a review from a team as a code owner September 6, 2022 09:12
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 6, 2022

Codecov Report

Merging #425 (cdf5426) into trunk (5fb8fc8) will not change coverage.
The diff coverage is n/a.

❗ Current head cdf5426 differs from pull request most recent head 698288d. Consider uploading reports for the commit 698288d to get more accurate results

@@           Coverage Diff           @@
##            trunk     #425   +/-   ##
=======================================
  Coverage   96.91%   96.91%           
=======================================
  Files          13       13           
  Lines         551      551           
  Branches      112      112           
=======================================
  Hits          534      534           
  Misses          9        9           
  Partials        8        8           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Copy Markdown
Member

@adiroiban adiroiban left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Much appreciated.

I will merge this as this is better than what we have in trunk.

Another option is not to have any example config inside README and just link to the docs.

But can be done in another PR.

@adiroiban adiroiban enabled auto-merge September 6, 2022 09:28
@adiroiban adiroiban merged commit 474e489 into twisted:trunk Sep 6, 2022
@jenshnielsen jenshnielsen deleted the fix_readme_typo branch September 6, 2022 09:28
bors bot added a commit to microsoft/Qcodes that referenced this pull request Sep 7, 2022
4580: Use Towncriers new config format r=jenshnielsen a=jenshnielsen

The old one is deprecated and this is less verbose and nicer.

Note that the format documented in the readme is actually wrong twisted/towncrier#425

4583: Validate pyproject.toml in precommit hook r=jenshnielsen a=jenshnielsen



Co-authored-by: Jens H. Nielsen <Jens.Nielsen@microsoft.com>
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.

Configuration of fragments in readme is incorrect

2 participants