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

Create src/README.md #239

Open
lautarodragan opened this issue Jul 19, 2018 · 0 comments
Open

Create src/README.md #239

lautarodragan opened this issue Jul 19, 2018 · 0 comments
Assignees

Comments

@lautarodragan
Copy link
Member

lautarodragan commented Jul 19, 2018

  • Explain architecture
  • Explain that each subfolder has its own README explaining the inner workings in more detail
  • Update main README.md with an Architecture section linking to this file
  • For src/extensions, link to sugarjs for arguments in favor of it and explanations. Also possibly mention how we may choose to drop this style in the future if we adopt Ramda, since, for example, myArray.filter(evenNumbers).toObject() reads better than arrayToObject(myArray.filter(evenNumbers)) for the same reasons listed in the pipeline operator proposal, but something like pipe(filter(isEven), arrayToObject)(myArray) could work.
@lautarodragan lautarodragan self-assigned this Jul 19, 2018
@lautarodragan lautarodragan added this to the Mainnet-launch milestone Jul 19, 2018
@geoffturk geoffturk assigned geoffturk and unassigned geoffturk Aug 7, 2018
@geoffturk geoffturk modified the milestones: Mainnet-launch, PML1 Nov 12, 2018
@geoffturk geoffturk removed this from the PML1 milestone Nov 29, 2018
@geoffturk geoffturk added ct-z and removed ct-z labels Nov 29, 2018
@geoffturk geoffturk added this to the PML1 milestone Nov 29, 2018
This was referenced Dec 4, 2018
@lautarodragan lautarodragan removed this from the PML1 milestone Jan 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants