Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Reword the sample config header to be less scary #4801

Merged
merged 5 commits into from
Mar 7, 2019

Conversation

ara4n
Copy link
Member

@ara4n ara4n commented Mar 5, 2019

No description provided.

@ara4n ara4n requested a review from richvdh March 5, 2019 00:01
@codecov
Copy link

codecov bot commented Mar 5, 2019

Codecov Report

Merging #4801 into develop will decrease coverage by 1.39%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop    #4801     +/-   ##
==========================================
- Coverage    75.12%   73.72%   -1.4%     
==========================================
  Files          340      340             
  Lines        34917    36511   +1594     
  Branches      5723     6192    +469     
==========================================
+ Hits         26231    26919    +688     
- Misses        7071     7913    +842     
- Partials      1615     1679     +64

Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

header looks good to me; thanks for clearing this up.

I've tweaked the newsfile to match that used for the original change, so that they can share a changelog entry come release time. Hopefully that will also stop the CI complaining about the missing trailing full-stop.

@richvdh
Copy link
Member

richvdh commented Mar 5, 2019

hah, except of course we need to actually regenerate the file. /me does so

@richvdh richvdh merged commit 8f4b9f5 into develop Mar 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants