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.

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

[BUG] standard linter throws error about one of its dependencies #628

Open
duncdrum opened this issue Jun 7, 2021 · 2 comments
Open

[BUG] standard linter throws error about one of its dependencies #628

duncdrum opened this issue Jun 7, 2021 · 2 comments
Assignees

Comments

@duncdrum
Copy link
Contributor

duncdrum commented Jun 7, 2021

To reproduce

run standard 'src/test/**/*.js' see d98c501
see standard/deglob#20
this should be renabled as part of the npm test command.

It might go away with more updates to dependencies i can't tell

Recording of runs on our old dasboard is working again
Screenshot 2021-06-07 at 23 59 43

There is now an additional pill on the readme, in addition to the link, so it's easier to find.

Context (please always complete the following information):

  • eXist-db Version: [e.g. 5.3.0-SNAPSHOT]
  • App Version: [e.g. 5.2.1-SNAPSHOT]
@duncdrum duncdrum added the bug label Jun 7, 2021
@duncdrum duncdrum self-assigned this Jun 7, 2021
@line-o
Copy link
Member

line-o commented Jun 8, 2021

standard linter runs locally without an issue.

@duncdrum
Copy link
Contributor Author

duncdrum commented Jun 8, 2021

i ve tested this with two node and standard versions locally and in all instances consistently reproduced the error we saw on CI. The deglob package is part of the standard org, and standard is its sole user in our repo. I expect there to be a version requirement that triggers, this, some combination of Standard x with node y. The gulp-standard plugin works without a hitch.

Its very well possible that once #629 is resolved, the issue will disappear.

duncdrum added a commit that referenced this issue Sep 29, 2021
bug still present
see #628
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants