README into functional-tests
ClosedPublic
Actions

Authored by jasonbcox on Jun 14 2018, 00:41.

Details

Reviewers

deadalnix
schancel

Group Reviewers

Restricted Project

Commits

rSTAGING1c970e085c09: Merged /test/functional/README into functional-tests
rABC1c970e085c09: Merged /test/functional/README into functional-tests

Summary

There was a goof in D1497 where I said I moved /test/functional/README but actually moved /test/README.
This diff merges /test/functional/README with what was /test/README turned functional-tests.md.
Completes T357, whereas D1497 actually completed T356.

Test Plan

read it

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jasonbcox created this revision.Jun 14 2018, 00:41

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 14 2018, 00:41

Build Bitcoin-ABC :: Diffs :: Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC :: Diffs :: Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B2740: Diff 4122.Jun 14 2018, 01:17

Rest of the content looks good.

doc/functional-tests.md
10 ↗	(On Diff #4122)	You added a wild period.
18 ↗	(On Diff #4122)	What's the rational for kicking this down a line?
160 ↗	(On Diff #4122)	Wild mid-dot appears.
163 ↗	(On Diff #4122)	What is Going On with The casing Here?
167 ↗	(On Diff #4122)	Please make this file use a consistent hard wrap.
199 ↗	(On Diff #4122)	Wrap is weird here again.

This revision now requires changes to proceed.Jun 15 2018, 12:11

jasonbcox marked 6 inline comments as done.Jun 15 2018, 17:14

jasonbcox added inline comments.

doc/functional-tests.md
10 ↗	(On Diff #4122)	Weird. This isn't even a period, but rather a bullet point. Not even sure how I managed to create it in the first place.
18 ↗	(On Diff #4122)	It doesn't match the rest of the doc having one line so long. As a result, it renders poorly in vim. I've fixed the wrapping in accordance with your other feedback.