ci010 / electron-vue-next

A starter template for using vue-next with the electron.
https://ci010.github.io/electron-vue-next/
191 stars 27 forks source link

ci(lint): Auto-fix code style issues #29

Closed cawa-93 closed 3 years ago

cawa-93 commented 3 years ago

I've worked on the code validation workflow a bit.

Made changes:

However, this does not work perfectly at all 100%. I noticed that in situations where auto-fix push the changes AND the code still has errors, the PR may not be displayed as untested by the linter.

ci010 commented 3 years ago

I think this could be optional, as it will automatically modify code on branch. I'm working on to make this repo as monorepo. So we can have create-electron-vue-app with options. You can add this as an option there.

I would like to use new npm 7 workspace, but it seems the new npm has the bug with optional dependencies... One way is we use lerna now until the npm fix its issue.