Make a doc that has the solidity style guide + the Chainlink style guide

Question

Make a doc that has the solidity style guide + the Chainlink style guide

PatrickAlphaC opened this issue 3 years ago · 5 comments

PatrickAlphaC commented 3 years ago

Describe the enhancement

Make a PR to the chainlink github with the style guide officially laid out.

PatrickAlphaC commented 2 years ago

Sure!

Answer 1 · 2022-06-21T18:11:05.000Z

How's this #434 ? Let me know if that needs some improvements or if you were thinking of something different altogether.

Also let me know if it's better I make a new PR since #434 is a bit of a mess. It has multiple commits cause I kept fixing the formatting. And it also has a typo fix for README.md

Answer 2 · 2022-07-06T01:27:18.000Z

Does this still have to be done? If so, where in Chainlink's GitHub should the PR be made? Cheers.

Answer 3 · 2022-07-06T02:20:23.000Z

Probably the chainlink core repo.

https://github.com/smartcontractkit/chainlink

I'd like someone from the Chainlink Labs org to do it though, to make sure the style is correct.

Answer 4 · 2022-07-06T17:12:48.000Z

Probably the chainlink core repo.

https://github.com/smartcontractkit/chainlink

I'd like someone from the Chainlink Labs org to do it though, to make sure the style is correct.

I see! Is it okay if I do one myself and make a PR there? Because that way, if necessary, it will be easier for someone from Chainlink Labs to verify/edit the guide. Cheers.