Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

See if I can fix crowdsourcing jslint pre-commit hook error #4223

Merged
merged 1 commit into from
Dec 2, 2021

Conversation

moyapchen
Copy link
Contributor

@moyapchen moyapchen commented Dec 1, 2021

Not sure why this is suddenly causing the jslint to fail when it didn't prior, but seems like a reasonable thing to fix.

Copypasting from parlai/mturk/package.json file of https://github.com/facebookresearch/ParlAI/pull/3362/files


Using this change to see if it fixes the stuff. Indeed, the jslint test no longer fails (though there's one in cleaninstall that I'm currently yak shaving...)

Not sure why this is suddenly causing the jslint to fail when it didn't prior, but seems like a reasonable thing to fix.

Copypasting from `parlai/mturk/package.json` file of https://github.com/facebookresearch/ParlAI/pull/3362/files
@moyapchen moyapchen changed the title See if I can fix crowdsourcing lint error See if I can fix crowdsourcing jslint pre-commit hook error Dec 1, 2021
@moyapchen moyapchen marked this pull request as ready for review December 1, 2021 20:48
@EricMichaelSmith
Copy link
Contributor

Hmm @moyapchen what's the jslint pre-commit hook error that you see without this change? If it's fine with @JackUrb it's fine with me

@moyapchen
Copy link
Contributor Author

@EricMichaelSmith See things in https://github.com/facebookresearch/ParlAI/actions/workflows/lint.yml

(A lot of it's my diffs, but if you scroll through, seems like other folks are hitting it too)

@JackUrb
Copy link
Contributor

JackUrb commented Dec 1, 2021

Seems alright to me as well, unsure what the sudden change was, but if this resolves it then LGTM

Copy link
Contributor

@EricMichaelSmith EricMichaelSmith left a comment

Choose a reason for hiding this comment

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

Sounds reasonable, thanks!

@moyapchen moyapchen merged commit d5ba66e into main Dec 2, 2021
@moyapchen moyapchen deleted the crowdsourcing_lint branch December 2, 2021 05:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants