Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Update names back to original for eslint-comments and no-only-tests#605

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
gracepark merged 1 commit intomainfromupdate-eslint-comments-no-only-tests-name
Jan 30, 2025

Conversation

gracepark
Copy link
Contributor

Updating names back to original for the plugin.

@CopilotCopilotAI review requested due to automatic review settingsJanuary 30, 2025 22:58
@graceparkgracepark requested a review froma team as acode ownerJanuary 30, 2025 22:58
Copy link

@CopilotCopilotAI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Tip: Copilot code review supports C#, Go, Java, JavaScript, Markdown, Python, Ruby and TypeScript, with more languages coming soon.Learn more

@@ -17,21 +17,21 @@ module.exports = {
},
plugins: {
prettierPlugin,
eslintComments,
'eslint-comments':eslintComments,
Copy link
Contributor

@jibrangjibrangJan 30, 2025
edited
Loading

Choose a reason for hiding this comment

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

Just curious. Why is necessary to revert to the original ones?

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

This way we can use the same names in dotcom. Or else we'd have to update the names in the monolith toeslintComments.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for clarifying!

Copy link
Contributor

@jibrangjibrang left a comment

Choose a reason for hiding this comment

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

LGTM

@graceparkgracepark merged commita40a2b9 intomainJan 30, 2025
7 checks passed
@graceparkgracepark deleted the update-eslint-comments-no-only-tests-name branchJanuary 30, 2025 23:02
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

Copilot code reviewCopilotCopilot left review comments

@jibrangjibrangjibrang approved these changes

@dgreifdgreifAwaiting requested review from dgreifdgreif is a code owner automatically assigned from github/web-systems-reviewers

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@gracepark@jibrang

[8]ページ先頭

©2009-2025 Movatter.jp