modules icon indicating copy to clipboard operation
modules copied to clipboard

[style] Add eslint rules for easier contribution

Open mrbarletta opened this issue 6 years ago • 2 comments

Hey team, botpress has linting rules .eslintrc and some scripts defined in package.json that makes contributing and styling a lot easier :)

I tried to copy those to botpress-modules and there are a lot of changes to do to comply with the same rules defined for botpress.

Should I do a PR adding the same .eslintrc found in botpress + the recommended fixes?

mrbarletta avatar Mar 16 '18 23:03 mrbarletta

Hey! That's something that I wanted to do forever and having the monorepo will help so much, as all repos will point to the root .eslintrc file!

I think two separate PRs would be best, the addition big the files will be merged instantly whereas the other PR will require me to compile & test all modules individually.

Thanks!!

On Fri, Mar 16, 2018, 7:22 PM Matias Barletta [email protected] wrote:

Hey team, botpress has linting rules .eslintrc and some scripts defined in package.json that makes contributing and styling a lot easier :)

I tried to copy those to botpress-modules and there are a lot of changes to do to comply with the same rules defined for botpress.

Should I do a PR adding the same .eslintrc found in botpress + the recommended fixes?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/botpress/modules/issues/5, or mute the thread https://github.com/notifications/unsubscribe-auth/ABQStDcuUQLK7i1h6zA5dzVN1KxkdESxks5tfEkpgaJpZM4Sujt5 .

slvnperron avatar Mar 16 '18 23:03 slvnperron

Also note that having prettier setup here would make your life much easier, as you can run a single command to restructure all files.

On Fri, Mar 16, 2018, 7:41 PM Sylvain Perron [email protected] wrote:

Hey! That's something that I wanted to do forever and having the monorepo will help so much, as all repos will point to the root .eslintrc file!

I think two separate PRs would be best, the addition big the files will be merged instantly whereas the other PR will require me to compile & test all modules individually.

Thanks!!

On Fri, Mar 16, 2018, 7:22 PM Matias Barletta [email protected] wrote:

Hey team, botpress has linting rules .eslintrc and some scripts defined in package.json that makes contributing and styling a lot easier :)

I tried to copy those to botpress-modules and there are a lot of changes to do to comply with the same rules defined for botpress.

Should I do a PR adding the same .eslintrc found in botpress + the recommended fixes?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/botpress/modules/issues/5, or mute the thread https://github.com/notifications/unsubscribe-auth/ABQStDcuUQLK7i1h6zA5dzVN1KxkdESxks5tfEkpgaJpZM4Sujt5 .

slvnperron avatar Mar 16 '18 23:03 slvnperron