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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update xo #838

Merged
merged 3 commits into from Mar 20, 2024
Merged

Update xo #838

merged 3 commits into from Mar 20, 2024

Conversation

voxpelli
Copy link
Collaborator

This is a straight up update and auto-fix

Only manual fix was the addition of this in one file:

// eslint-disable-next-line unicorn/prevent-abbreviations

@voxpelli voxpelli self-assigned this Mar 20, 2024
@voxpelli
Copy link
Collaborator Author

  • Added npm-run-all2 as then I got feedback as to which of the five scripts if was that was failing (and it runs the tests in parallel which means quicker execution)
  • Stopped running CI tests on pushes to non-main branches, they will be the source of PR:s anyway and as such will get CI run as part of the PR as well, which previously resulted in duplicated tests

This was referenced Mar 20, 2024
Merged
Closed
sindresorhus
sindresorhus requested changes Mar 20, 2024
View reviewed changes
source/subtract.d.ts Outdated Show resolved Hide resolved
@voxpelli voxpelli mentioned this pull request Mar 20, 2024
Merged
@sindresorhus sindresorhus merged commit 95ac6df into main Mar 20, 2024
6 checks passed
@sindresorhus sindresorhus deleted the update-xo branch March 20, 2024 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sindresorhus sindresorhus Awaiting requested review from sindresorhus

Assignees

@voxpelli voxpelli

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@voxpelli @sindresorhus