search

wegue-oss / wegue

Template and components for webmapping applications with OpenLayers and Vue.js
BSD 2-Clause "Simplified" License
93 stars 41 forks source link

Explicitly call 'npm run lint' in CI test pipeline #375

Closed chrismayer closed 2 months ago

chrismayer commented 2 months ago

This starts the linting process (npm run lint) explicitly in the CI test pipeline because the implicit lint call in the npm run test command does not break the pipeline, even if there are linting errors. So possible linting errors land in the Wegue code base if not discovered by human reviewers, which leads to a broken build.

chrismayer commented 2 months ago

Thanks for the review @JakobMiksch.