Skip to content

Use the npm lint script on CI.#604

Closed
XhmikosR wants to merge 1 commit into
npm:latestfrom
XhmikosR:lint
Closed

Use the npm lint script on CI.#604
XhmikosR wants to merge 1 commit into
npm:latestfrom
XhmikosR:lint

Conversation

@XhmikosR
Copy link
Copy Markdown
Contributor

@XhmikosR XhmikosR commented Dec 16, 2019

This should be more consistent since it should use the specified standard version and not the latest one.

@XhmikosR XhmikosR requested a review from a team as a code owner December 16, 2019 15:10
@isaacs
Copy link
Copy Markdown
Contributor

isaacs commented Dec 16, 2019

This should be more consistent since it should use the specified standard version and not the latest one.

Note: npx will use the version installed in node_modules if it exists. It only pulls the latest if there isn't something else specified locally. But abstracting it out in case we change our linter at some point in the future is not a bad idea anyway, so at least it's only defined in one place.

@XhmikosR
Copy link
Copy Markdown
Contributor Author

XhmikosR commented Dec 16, 2019

Oh, cool, I wasn't aware of that. But yeah, this does feel better to me, more consistent.

@mikemimik mikemimik added this to the OSS - Sprint 1 milestone Dec 17, 2019
@mikemimik mikemimik added Community Enhancement new feature or improvement semver:patch semver patch level for changes labels Dec 19, 2019
@mikemimik mikemimik self-assigned this Dec 19, 2019
mikemimik
mikemimik approved these changes Dec 19, 2019
View reviewed changes
Copy link
Copy Markdown
Contributor

@mikemimik mikemimik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mikemimik
Copy link
Copy Markdown
Contributor

mikemimik commented Dec 19, 2019

nit: We might want to address the failing tests

@XhmikosR
Copy link
Copy Markdown
Contributor Author

XhmikosR commented Dec 22, 2019

Might be worth bumping standard to the version that supports the Node.js versions you guys support so that everything is aligned.

@XhmikosR
Use the npm lint script on CI.
562cfe0 
This should be more consistent since it should use the specified standard version and not the latest one.
@mikemimik mikemimik added the Release 6.x work is associated with a specific npm 6 release label Jan 21, 2020
@mikemimik mikemimik closed this in 7852c0c Jan 28, 2020
@XhmikosR XhmikosR deleted the lint branch January 29, 2020 07:01
@Theo13111 Theo13111 mentioned this pull request Aug 10, 2023
Open
Jah-yee pushed a commit to Jah-yee/cli that referenced this pull request Apr 16, 2026
@jpoehnelt @jpoehnelt-bot @googleworkspace-bot
fix(ci): skip publish-skills on fork PRs (npm#604)
4663021 
* fix(ci): skip publish-skills on fork PRs where secrets are unavailable

* chore: regenerate skills [skip ci]

---------

Co-authored-by: jpoehnelt-bot <[email protected]>
Co-authored-by: googleworkspace-bot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@mikemimik mikemimik mikemimik approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@mikemimik mikemimik

Labels

Enhancement new feature or improvement Release 6.x work is associated with a specific npm 6 release semver:patch semver patch level for changes

Projects

None yet

Milestone

OSS - Sprint 2

Development

Successfully merging this pull request may close these issues.

4 participants

@XhmikosR @isaacs @mikemimik @darcyclarke