Page MenuHomePhabricator

[RPC docs] Remove unused index files
ClosedPublic

Authored by jasonbcox on Jun 15 2020, 21:42.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Commits
rABC148c4f7f071e: [RPC docs] Remove unused index files
Summary

These index files are not used and contain nothing useful. Now that D7130 and D7131 are landed, we know these index files will never be used.

Test Plan
ninja doc-rpc

Verify doc/rpc/en/0.22.0/index.html and doc/rpc/en/0.22.0/rpc/index.html are no longer generated.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
rm-rpc-doc-index
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 12368
Build 24920: Build Diffbuild-without-wallet · build-diff · build-clang-tidy · build-clang-10
Build 24919: arc lint + arc unit

Event Timeline

It'd be probably preferable to use the template to generate something useful rather than removing this.

It was conflicting with local tests of using nginx to generate a quick index for us prior to having a nicely-formatted index. But you're right, there's probably value in having this if we can fix it up.

Worst case and short term, we can remove the index files before we deploy so that a usable index is always present...

Rebase. See updated summary.

jasonbcox edited the test plan for this revision. (Show Details)
This revision is now accepted and ready to land.Aug 13 2020, 16:59
This revision was automatically updated to reflect the committed changes.