l3mpire / lemverse

The first-ever coworking metaverse
https://app.lemverse.com
GNU Affero General Public License v3.0
149 stars 40 forks source link

Some files do not respect the linter #151

Open Flowtter opened 1 year ago

Flowtter commented 1 year ago

Some files do not respect the current linter configuration (eslint). https://github.com/l3mpire/lemverse/blob/19a80c88c939892a865bc2f82d0182ec944c8e66/core/client/scenes/scene-editor.js#L114-L119

The expected behavior would be:

          Zones.update(zoneId, {
            $set: {
              x1: startPosition.x | 0,
              y1: startPosition.y | 0,
              x2: endPosition.x | 0,
              y2: endPosition.y | 0,
            }
          });

I'm thinking about making a PR with all the fixes, but that might create a lot of merge conflicts in the pending pull requests.

Would you allow us to lint correctly the files we're editing in our pull request, or would you rather do one single pull request/commit to fix all the project?

Adding a GitHub action to check the linting of the code would also probably be a good idea.

ramnes commented 1 year ago

You mean eslint? FWIW I've added editorconfig-checker to the development dependencies in #128, so if we add a GitHub action (and we should!) let's not forget to check against all linters.

Also I think pre-commits hooks would be just as important as a GitHub action. :)