Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CI: run lint once #3621

Closed
2 tasks done
Eomm opened this issue Jan 15, 2022 · 1 comment 路 Fixed by #3623
Closed
2 tasks done

CI: run lint once #3621

Eomm opened this issue Jan 15, 2022 · 1 comment 路 Fixed by #3623
Labels
chore Small changes or internal project maintenance good first issue Good for newcomers test Issue or pr related to our testing infrastructure.

Comments

@Eomm
Copy link
Member

Eomm commented Jan 15, 2022

Prerequisites

  • I have written a descriptive issue title
  • I have searched existing issues to ensure the feature has not already been requested

馃殌 Feature Proposal

Refactor the package.json scripts and the CI to run the linter only once.
(now it runs for every node version and OS)

This will:

  • speed up a bit the pipeline
  • improve the readability

Motivation

This output is confusing:

image

Example

No response
@Eomm Eomm added good first issue Good for newcomers chore Small changes or internal project maintenance test Issue or pr related to our testing infrastructure. labels Jan 15, 2022
@jsumners
Copy link
Member

馃挴

@darkgl0w darkgl0w mentioned this issue Jan 16, 2022
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
chore Small changes or internal project maintenance good first issue Good for newcomers test Issue or pr related to our testing infrastructure.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore (ci): run lint only once
2 participants
@jsumners @Eomm