Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

peng/fix docs base directory #2295

Merged
merged 1 commit into from
Aug 31, 2018
Merged

Conversation

nylira
Copy link
Contributor

@nylira nylira commented Aug 29, 2018

  • Updated CHANGELOG_PENDING.md

This is a mirror of this PR cosmos/cosmos-sdk#2174, which fixes the root directory and rendering of the documentation website (tendermint.com/docs).

@@ -2,23 +2,21 @@ module.exports = {
title: "Tendermint Core",
description: "Documentation for Tendermint Core",
dest: "./dist/docs",
base: "/",
base: "/docs/",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we also remove this prefix from links in .md files?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@faboweb I don't think that's necessary, right?

Copy link

Choose a reason for hiding this comment

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

can you point me to an example on the website?

Copy link
Contributor Author

@nylira nylira Aug 29, 2018

Choose a reason for hiding this comment

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

@melekes this base: "/docs/" refers to the documentation website being run under a subdirectory of the Tendermint website. I do believe there are links that need to be fixed in the Tendermint docs to ensure they work on the docs website. But that probably belongs in a different PR (@jb @zramsay are the ones who would know more). This PR is a high priority merge for develop and master. @greg-szabo

Copy link
Contributor

Choose a reason for hiding this comment

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

Then we should ask @ebuchman to include it into #2261

Copy link
Contributor

Choose a reason for hiding this comment

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

Seems theres a few docs related things on develop that aren't on master. Can we get a separate PR against master that just copies these over, or at least whatever parts we need?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can we get someone assigned to this?

Copy link
Contributor

Choose a reason for hiding this comment

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

Is #2336 enough?

@@ -26,6 +26,7 @@ FEATURES:

IMPROVEMENTS:
- [docs] Lint documentation with `write-good` and `stop-words`.
- [docs] Fix base directory of documentation website.
Copy link
Contributor

Choose a reason for hiding this comment

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

no need in changelog entry

@codecov-io
Copy link

codecov-io commented Aug 29, 2018

Codecov Report

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

@@             Coverage Diff             @@
##           develop    #2295      +/-   ##
===========================================
- Coverage    60.92%   60.89%   -0.03%     
===========================================
  Files          196      196              
  Lines        16173    16166       -7     
===========================================
- Hits          9853     9844       -9     
+ Misses        5464     5463       -1     
- Partials       856      859       +3
Impacted Files Coverage Δ
libs/db/go_level_db.go 65% <0%> (-1.67%) ⬇️
blockchain/pool.go 65.73% <0%> (-0.7%) ⬇️
p2p/pex/pex_reactor.go 73.5% <0%> (-0.67%) ⬇️
mempool/mempool.go 71.49% <0%> (-0.13%) ⬇️
consensus/reactor.go 73.55% <0%> (ø) ⬆️
consensus/state.go 76.82% <0%> (+0.24%) ⬆️
privval/priv_validator.go 69.88% <0%> (+0.44%) ⬆️

@ebuchman ebuchman merged commit 5ecdfac into tendermint:develop Aug 31, 2018
@nylira nylira deleted the peng/fix-docs branch September 5, 2018 04:05
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.

None yet

5 participants