Skip to content

Commit da3ba7e

Browse files
authored
Fix/codacy improvements (#1075)
* more md lint * more md linting
1 parent 196e413 commit da3ba7e

File tree

3 files changed

+60
-55
lines changed

3 files changed

+60
-55
lines changed

README-dev.md

+36-29
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22

33
Coda is a new cryptocurrency protocol with a lightweight, constant sized blockchain.
44

5-
* [Developer homepage](https://codaprotocol.com/code.html)
6-
* [Roadmap](https://github.com/orgs/CodaProtocol/projects/1)
7-
* [Repository Readme](README.md)
5+
* [Developer homepage](https://codaprotocol.com/code.html)
6+
* [Roadmap](https://github.com/orgs/CodaProtocol/projects/1)
7+
* [Repository Readme](README.md)
88

99
If you haven't seen it yet, [CONTRIBUTING.md](CONTRIBUTING.md) has information
1010
about our development process and how to contribute. If you just want to build
@@ -29,27 +29,34 @@ Now you'll have a `src/_build/codaclient.deb` ready to install on Ubuntu or Debi
2929
### Developer Setup
3030

3131
#### Install or have Ubuntu 18
32+
3233
* [VM Images](https://www.osboxes.org/ubuntu/)
3334

3435
#### Setup Docker CE on Ubuntu
35-
* [Ubuntu Setup Instructions](https://docs.docker.com/install/linux/docker-ce/ubuntu/)
36+
37+
* [Ubuntu Setup Instructions](https://docs.docker.com/install/linux/docker-ce/ubuntu/)
3638

3739
#### Toolchain docker image
38-
* Pull down developer container image (~2GB download, go stretch your legs)\
40+
41+
* Pull down developer container image (~2GB download, go stretch your legs)
42+
3943
`docker pull codaprotocol/coda:toolchain-e11592718bee89d2a4facfa7ca209844fa7b140c`
4044

41-
* Apply local customizatins\
45+
* Apply local customizations
46+
4247
`make docker`
4348

44-
* Start developer container\
49+
* Start developer container
50+
4551
`make container`
4652

47-
* Start a build (go stretch your arms)\
53+
* Start a build (go stretch your arms)
54+
4855
`make USEDOCKER=TRUE build`
4956

5057
#### Customizing your dev environment for autocomplete/merlin
51-
* If you use vim, add this snippet in your vimrc to use merlin.\
52-
(REMEMBER to change the HOME directory to match yours)
58+
59+
* If you use vim, add this snippet in your vimrc to use merlin. (REMEMBER to change the HOME directory to match yours)
5360

5461
```bash
5562
let s:ocamlmerlin="/Users/USERNAME/.opam/4.07/share/merlin"
@@ -58,15 +65,14 @@ execute "set rtp+=".s:ocamlmerlin."/vimbufsync"
5865
let g:syntastic_ocaml_checkers=['merlin']
5966
```
6067

61-
* In your home directory `opam init`
62-
* In this shell, `eval $(opam config env)`
63-
* Now `/usr/bin/opam install merlin ocp-indent core async ppx_jane ppx_deriving` (everything we depend on, that you want autocompletes for) for doc reasons
64-
* Make sure you have `au FileType ocaml set omnifunc=merlin#Complete` in your vimrc
65-
* Install an auto-completer (such as YouCompleteMe) and a syntastic (such syntastic or ALE)
66-
67-
* If you use vscode, you might like these extensions
68-
* [OCaml and Reason IDE](https://marketplace.visualstudio.com/items?itemName=freebroccolo.reasonml)
69-
* [Dune](https://marketplace.visualstudio.com/items?itemName=maelvalais.dune)
68+
* In your home directory `opam init`
69+
* In this shell, `eval $(opam config env)`
70+
* Now `/usr/bin/opam install merlin ocp-indent core async ppx_jane ppx_deriving` (everything we depend on, that you want autocompletes for) for doc reasons
71+
* Make sure you have `au FileType ocaml set omnifunc=merlin#Complete` in your vimrc
72+
* Install an auto-completer (such as YouCompleteMe) and a syntastic (such syntastic or ALE)
73+
* If you use vscode, you might like these extensions
74+
* [OCaml and Reason IDE](https://marketplace.visualstudio.com/items?itemName=freebroccolo.reasonml)
75+
* [Dune](https://marketplace.visualstudio.com/items?itemName=maelvalais.dune)
7076

7177
## Using the makefile
7278

@@ -77,13 +83,13 @@ You should probably use `USEDOCKER=TRUE` unless you've done the [building withou
7783

7884
These are the most important `make` targets:
7985

80-
* `kademlia`: build the kademlia helper
81-
* `build`: build everything
82-
* `docker`: build the container
83-
* `container`: restart the development container (or start it if it's not yet)
84-
* `dev`: does `docker`, `container`, and `build`
85-
* `test`: run the tests
86-
* `web`: build the website, including the state explorer
86+
* `kademlia`: build the kademlia helper
87+
* `build`: build everything
88+
* `docker`: build the container
89+
* `container`: restart the development container (or start it if it's not yet)
90+
* `dev`: does `docker`, `container`, and `build`
91+
* `test`: run the tests
92+
* `web`: build the website, including the state explorer
8793

8894
We use the [dune](https://github.com/ocaml/dune/) buildsystem for our OCaml code.
8995

@@ -101,7 +107,7 @@ you need, you run `opam switch import src/opam.export`.
101107
Some of our dependencies aren't taken from `opam`, and aren't integrated
102108
with `dune`, so you need to add them manually:
103109

104-
* `opam pin add src/external/ocaml-sodium`
110+
* `opam pin add src/external/ocaml-sodium`
105111

106112
There are a variety of C libraries we expect to be available in the system.
107113
These are also listed in the dockerfiles.
@@ -115,7 +121,7 @@ in the form of `dune exec coda -- integration-tests $SOME_TEST`.
115121

116122
You might see a build error like this:
117123

118-
```
124+
```text
119125
Error: Files external/digestif/src-c/.digestif_c.objs/digestif.cmx
120126
and external/digestif/src-c/.digestif_c.objs/rakia.cmx
121127
make inconsistent assumptions over implementation Rakia
@@ -127,6 +133,7 @@ Here, the offending path is `external/digestif/src-c/.diestif_c.objs`.
127133
## Docker Image Family Tree
128134

129135
Container Stages:
130-
* Stage 0: Initial Image [ocaml/opam2:debian-9-ocaml-4.07](https://hub.docker.com/r/ocaml/opam2/) (opam community image, ~880MB)
136+
137+
* Stage 0: Initial Image [ocaml/opam2:debian-9-ocaml-4.07](https://hub.docker.com/r/ocaml/opam2/) (opam community image, ~880MB)
131138
* Stage 1: [coda toolchain](https://github.com/CodaProtocol/coda/blob/master/dockerfiles/Dockerfile-toolchain) (built by us, stored on docker hub, ~2GB compressed)
132139
* Stage 2: [codabuilder](https://github.com/CodaProtocol/coda/blob/master/dockerfiles/Dockerfile) (built with `make docker`, used with `make build`, ~2GB compressed)

README.md

+6-5
Original file line numberDiff line numberDiff line change
@@ -8,19 +8,20 @@ We have a [Discord server]( https://discord.gg/ShKhA7J)! Please come by if you
88
need help or have questions. You might also be interested in the [OCaml
99
Discord](https://discordapp.com/invite/cCYQbqN), for general OCaml help.
1010

11-
# Table of Contents
11+
## Table of Contents
1212

1313
* [Coda homepage](https://codaprotocol.com/)
1414
* [Roadmap](https://github.com/orgs/CodaProtocol/projects/1)
1515
* [Developer homepage](https://codaprotocol.com/code.html)
1616
* [Developer readme](README-dev.md)
1717
* [Compiling from source and and running a node](docs/demo.md)
1818

19-
# Learn more
20-
* [Directory structure](DIRECTORY_STRUCTURE.md)
21-
* [Lifecycle of a transaction](docs/lifecycle_of_a_transaction_lite.md)
19+
## Learn more
2220

23-
# License
21+
* [Directory structure](DIRECTORY_STRUCTURE.md)
22+
* [Lifecycle of a transaction](docs/lifecycle_of_a_transaction_lite.md)
23+
24+
## License
2425

2526
This repository is distributed under the terms of the Apache 2.0 license,
2627
available in the LICENSE fail and online at

rfcs/0000-template.md

+18-21
Original file line numberDiff line numberDiff line change
@@ -1,47 +1,44 @@
1-
# Summary
1+
## Summary
22
[summary]: #summary
33

44
One paragraph explanation of the feature.
55

6-
# Motivation
6+
## Motivation
7+
78
[motivation]: #motivation
89

910
Why are we doing this? What use cases does it support? What is the expected outcome?
1011

11-
# Detailed design
12+
## Detailed design
13+
1214
[detailed-design]: #detailed-design
1315

1416
This is the technical portion of the RFC. Explain the design in sufficient detail that:
1517

16-
- Its interaction with other features is clear.
17-
- It is reasonably clear how the feature would be implemented.
18-
- Corner cases are dissected by example.
18+
* Its interaction with other features is clear.
19+
* It is reasonably clear how the feature would be implemented.
20+
* Corner cases are dissected by example.
1921

20-
# Drawbacks
22+
## Drawbacks
2123
[drawbacks]: #drawbacks
2224

2325
Why should we *not* do this?
2426

25-
# Rationale and alternatives
27+
## Rationale and alternatives
2628
[rationale-and-alternatives]: #rationale-and-alternatives
2729

28-
- Why is this design the best in the space of possible designs?
29-
- What other designs have been considered and what is the rationale for not choosing them?
30-
- What is the impact of not doing this?
30+
* Why is this design the best in the space of possible designs?
31+
* What other designs have been considered and what is the rationale for not choosing them?
32+
* What is the impact of not doing this?
3133

32-
# Prior art
34+
## Prior art
3335
[prior-art]: #prior-art
3436

3537
Discuss prior art, both the good and the bad, in relation to this proposal.
36-
A few examples of what this can include are:
3738

38-
# Unresolved questions
39+
## Unresolved questions
3940
[unresolved-questions]: #unresolved-questions
4041

41-
- What parts of the design do you expect to resolve through the RFC process
42-
before this gets merged?
43-
- What parts of the design do you expect to resolve through the implementation
44-
of this feature before merge?
45-
- What related issues do you consider out of scope for this RFC that could be
46-
addressed in the future independently of the solution that comes out of this
47-
RFC?
42+
* What parts of the design do you expect to resolve through the RFC process before this gets merged?
43+
* What parts of the design do you expect to resolve through the implementation of this feature before merge?
44+
* What related issues do you consider out of scope for this RFC that could be addressed in the future independently of the solution that comes out of this RFC?

0 commit comments

Comments
 (0)