Skip to content

changelog: Add links everywhere, and script to auto-add links.#2341

Closed
ValarDragon wants to merge 1 commit intodevelopfrom
dev/changelog
Closed

changelog: Add links everywhere, and script to auto-add links.#2341
ValarDragon wants to merge 1 commit intodevelopfrom
dev/changelog

Conversation

@ValarDragon
Copy link
Contributor

@ValarDragon ValarDragon commented Sep 5, 2018

  • Updated all relevant documentation in docs - should I document this anywhere? The comments in the script seem sufficient, unless we have another docs page for scripts
  • Updated all code comments where relevant
  • Updated CHANGELOG_PENDING.md

@ebuchman
Copy link
Contributor

ebuchman commented Sep 5, 2018

OMG this is amazing thank you !

@ebuchman
Copy link
Contributor

ebuchman commented Sep 5, 2018

Can we get just the script against #2340 please ? I made more changes there - will run it my self when done editing the changelog

@ValarDragon
Copy link
Contributor Author

Sure!

@ValarDragon
Copy link
Contributor Author

Closing in favor of #2342

@ValarDragon ValarDragon closed this Sep 5, 2018
@ValarDragon ValarDragon deleted the dev/changelog branch September 5, 2018 23:00
@codecov-io
Copy link

Codecov Report

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

@@             Coverage Diff             @@
##           develop    #2341      +/-   ##
===========================================
- Coverage    60.94%   60.93%   -0.02%     
===========================================
  Files          197      197              
  Lines        16302    16302              
===========================================
- Hits          9936     9934       -2     
  Misses        5503     5503              
- Partials       863      865       +2
Impacted Files Coverage Δ
libs/autofile/autofile.go 69.69% <0%> (-6.07%) ⬇️
p2p/peer.go 57.95% <0%> (ø) ⬆️
consensus/reactor.go 73.02% <0%> (+0.26%) ⬆️

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