Skip to content

feat(eslint-plugin): include docs URL in lint message#3944

Merged
brandonroberts merged 1 commit intomainfrom
eslint-log-url
Jul 11, 2023
Merged

feat(eslint-plugin): include docs URL in lint message#3944
brandonroberts merged 1 commit intomainfrom
eslint-log-url

Conversation

@timdeschryver
Copy link
Member

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Closes #3942

What is the new behavior?

URL to docs is included within the message.

image

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@netlify
Copy link

netlify bot commented Jul 5, 2023

Deploy Preview for ngrx-io ready!

Name Link
🔨 Latest commit 62bd7a0
🔍 Latest deploy log https://app.netlify.com/sites/ngrx-io/deploys/64a5ade1598e5e0008cf71d1
😎 Deploy Preview https://deploy-preview-3944--ngrx-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@markostanimirovic markostanimirovic changed the title feat: include docs URL in lint message feat(eslint-plugin): include docs URL in lint message Jul 11, 2023
@brandonroberts brandonroberts merged commit a1576de into main Jul 11, 2023
@brandonroberts brandonroberts deleted the eslint-log-url branch July 11, 2023 12:20
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

Successfully merging this pull request may close these issues.

@ngrx/eslint-plugin (DX) - print links to broken ngrx eslint rules in output

3 participants