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

S/fixes and suggestions #1

Merged
merged 2 commits into from
Jan 20, 2021
Merged

S/fixes and suggestions #1

merged 2 commits into from
Jan 20, 2021

Conversation

stevenweber
Copy link

No description provided.

@stevenweber stevenweber requested a review from a team January 19, 2021 22:31
@ebenoist
Copy link

primary
?

I thought the new default was main? Or is that how they have their repo setup?

@stevenweber
Copy link
Author

primary
?

I thought the new default was main? Or is that how they have their repo setup?

That's how the source repo was configured https://github.com/muenzpraeger/eslint-plugin-inclusive-language/branches

const RULE_CATEGORY = 'Language';
const RULE_DESCRIPTION = 'highlights where non-inclusive language is used';
// eslint-disable-next-line no-unused-vars
const SUGGESTION_MESSAGE = "Replace word '{{word}}'' with '{{suggestion}}.'"; // Currently not in use
const SUGGESTION_MESSAGE = "Replace word '{{word}}' with '{{suggestion}}.'"; // Currently not in use

Choose a reason for hiding this comment

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

Is this comment still valid?

Copy link
Author

Choose a reason for hiding this comment

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

It's not!

suggestion: result[0].suggestion
};
const { word, explanation } = result;
// backwards-compatibility with singular suggestions

Choose a reason for hiding this comment

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

fancy

@@ -117,6 +142,9 @@ module.exports = {
Identifier(node) {
validateIfInclusive(context, node, node.name);
},
JSXIdentifier(node) {

Choose a reason for hiding this comment

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

👍

@ebenoist
Copy link

Nice work!

@stevenweber stevenweber force-pushed the s/fixes-and-suggestions branch from 7ae6b35 to a98c1c2 Compare January 20, 2021 15:21
@stevenweber stevenweber merged commit bf0e827 into primary Jan 20, 2021
@stevenweber stevenweber deleted the s/fixes-and-suggestions branch January 20, 2021 15:31
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.

2 participants