The Wayback Machine - https://web.archive.org/web/20210620194057/https://github.com/mrdoob/three.js/pull/21652
Skip to content

/ three.js

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.

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

Rename test-lint script to lint #21652

Merged
merged 3 commits into from Apr 14, 2021
Merged

Rename test-lint script to lint #21652

merged 3 commits into from Apr 14, 2021

Conversation

@marcofugaro
Copy link
Contributor

@marcofugaro marcofugaro commented Apr 14, 2021

Related issue: -

Description

Rename the script test-lint to just lint for consistency with other scripts.

Also use the existing scripts in the lint-fix scripts instead of duplicating the command.
marcofugaro added 3 commits Apr 14, 2021
@marcofugaro
package.json Rename test-lint script to lint
77c51d9
@marcofugaro
Update lint script calls
49f0021
@marcofugaro
lint-fix: Use existing scripts
9109308
@Mugen87
Mugen87 approved these changes Apr 14, 2021
View changes
@mrdoob mrdoob added this to the r128 milestone Apr 14, 2021
@mrdoob mrdoob merged commit 952441f into mrdoob:dev Apr 14, 2021
8 checks passed
8 checks passed
@github-actions
Linting
Details
@github-actions
Unit testing
Details
@github-actions
E2E testing (0)
Details
@github-actions
E2E testing (1)
Details
@github-actions
E2E testing (2)
Details
@github-actions
E2E testing (3)
Details
@github-actions
Ready for release
Details
@lgtm-com
LGTM analysis: JavaScript No new or fixed alerts
Details
@mrdoob
Copy link
Owner

@mrdoob mrdoob commented Apr 14, 2021

Thanks!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants
@marcofugaro @mrdoob @Mugen87