Update link lint rule to skip Link with className

[khiga8]

Fixes: #192
Related: https://github.com/github/github/pull/328018

This PR modifies the a11y-link-in-text-block rule to avoid flagging Link in text block if it also has the className on it.

The a11y-link-in-text-block is aimed at flagging Link that do not have distinguishing styles on it, and encourages the use of the inline prop to underline the link in such scenarios.

Sometimes, the Link has a className attribute on it to style it differently with CSS classes. In such cases, the rule should skip the Link because it's possible the CSS class provides sufficient styling. However, the linter cannot actually determine whether the CSS class provides sufficient styling, so this errs on the side of caution and skips Link with className entirely to avoid false positive flags.

Reviewers

I'm open to your feedback on whether this is the right approach to take, or if there are others I'm not considering.

[changeset-bot]

🦋 Changeset detected

Latest commit: 8477ca2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
eslint-plugin-primer-react	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR