Skip to content
#

linter

Here are 1,348 public repositories matching this topic...

rome
bradydowling
bradydowling commented Aug 12, 2020

When I ran rome check I got the following output:


 node_modules/deep-freeze/package.json:50:14 parse/spdxLicense ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Unknown license public

  ℹ The SPDX registry is used to ensure valid and legal licenses. See https://spdx.org/licenses/ for more information.

━━━━━━━━━━━━━━━━━━━━━━━━━
dmytro-savochkin
dmytro-savochkin commented Aug 9, 2020

I think Style/IfUnlessModifier doesn't correctly handle the case when there is a first-line comment (right after the if keyword) AND some code right after the end keyword. If you correct this code with rubocop -a it would put the comment right after the end keyword, before the code that was there previously. This basically makes the code invalid after correction.

This probably can be

stylelint
silverwind
silverwind commented Jul 28, 2020

Clearly describe the bug

currentcolor (all lowercase) should be acceptable casing. CSS Color Module Level 4 defines it like that. Info from above link:

Multi-word keywords in CSS usually separate their component words with hyphens. currentcolor doesn’t, because it was originally introduced in SVG as a special attribute value spel

golangci-lint
DarthHater
DarthHater commented Aug 18, 2020

Hello!

We saw you are using nancy, and we are humbled! I thought I'd reach out and let you know that we will have a 1.0.0 release coming soon, and usage is going to change slightly. To avoid disruption to your project, rather than using latest on your Docker image tag, I'd suggest using v0.3 or v0 for now, and then moving back to latest once you've had some time to digest the changes.

Improve this page

Add a description, image, and links to the linter topic page so that developers can more easily learn about it.

Curate this topic

Add this topic to your repo

To associate your repository with the linter topic, visit your repo's landing page and select "manage topics."

Learn more

You can’t perform that action at this time.