redux-offline / tools

MIT License
5 stars 3 forks source link

Sorting Lerna / Package Deps #1

Closed calumpeak closed 6 years ago

calumpeak commented 6 years ago

Just going through the monorepo - I'm fairly familiar with Lerna as I use it for a large project currently. There's some changes I'm going to raise to sort things out based on a PR that didn't make it in last time:

I would also like to standardise commit messages, how do we feel about an implementation of commitizen to ensure we have a good/descriptive commit history from all contributors? This is something I think should also be considered for offline core as well.

Thoughts?

sorodrigo commented 6 years ago

I agree that Eslint and Prettier should be in the root, but I think Babel, Flow should be there too. Check out the React monorepo they have their build dependencies at top level and their per package dependencies. Also the docs advises to do this.

I might be wrong, but I think we could use a global config and just override it locally if needed no?

About the commitizen suggestion I have tried it before in other projects, and found it to be more friction than help. I would prefer not including this, it could act as a wall against contributors.

calumpeak commented 6 years ago

Yeah that sounds about right - I'll sort out the deps that way :)

And no worries on cz, was just a suggestion 👍