Skip to content
This repository has been archived by the owner on Jul 27, 2021. It is now read-only.

Opinionated style for dxos modules. #1

Merged
merged 8 commits into from
Nov 13, 2019
Merged

Opinionated style for dxos modules. #1

merged 8 commits into from
Nov 13, 2019

Conversation

tinchoz49
Copy link
Contributor

@tinchoz49 tinchoz49 commented Nov 12, 2019

This PR show an example of an opinionated style:

  • Semistandard for lint: Most of the projects that we are using or taking inspiration like Dat, IPFS, Scuttlebut and WebTorrent use https://standardjs.com and it would nice to follow that path but I don't know if the wireline team is prepared to work without semicolons, that's why exists semistandard to provide Standard with semicolons. At least we can be close to the community in that way.
  • Travis for CI: Travis give us an easy way to deploy our open source projects and again is used by the entire community of Dat and IPFS.
  • Jest coverage + coveralls.
  • Standard Readme: A way to organize the readme. Use it by IPFS and Geut.

richburdon
richburdon approved these changes Nov 12, 2019
@@ -2,6 +2,11 @@

> A consistent store for your hypercore feeds.

[![Build Status](https://travis-ci.com/dxos/feed-store.svg?branch=tinchoz49-style)](https://travis-ci.com/dxos/feed-store)
[![Coverage Status](https://coveralls.io/repos/github/dxos/feed-store/badge.svg?branch=tinchoz49-style)](https://coveralls.io/github/dxos/feed-store?branch=tinchoz49-style)
[![js-semistandard-style](https://img.shields.io/badge/code%20style-semistandard-brightgreen.svg?style=flat-square)](https://github.com/standard/semistandard)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is going to be really painful but it's the right decision.

I'm just sad that so many people on the internet are wrong.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

at least we are going to have something in common with these companies: https://standardjs.com/#who-uses-javascript-standard-style

@tinchoz49 tinchoz49 merged commit 156cba3 into master Nov 13, 2019
@tinchoz49 tinchoz49 deleted the tinchoz49-style branch November 13, 2019 02:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants