+
Skip to content
This repository was archived by the owner on Aug 31, 2023. It is now read-only.

feat(rome_js_analyze): noDuplicateClassMembers #4130

Closed
wants to merge 1 commit into from
Closed

feat(rome_js_analyze): noDuplicateClassMembers #4130

wants to merge 1 commit into from

Conversation

Conaclos
Copy link
Contributor

@Conaclos Conaclos commented Jan 1, 2023

Summary

Closes #4129

Test Plan

ESLint's unit test included. Unit tests were also added for covering accessor.

Documentation

  • The PR requires documentation
  • I will create a new PR to update the documentation

@Conaclos Conaclos requested review from leops, ematipico, xunilrj and a team as code owners January 1, 2023 18:58
@netlify
Copy link

netlify bot commented Jan 1, 2023

Deploy Preview for docs-rometools ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 913625f
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/63b1dd1c6974f70009db263c
😎 Deploy Preview https://deploy-preview-4130--docs-rometools.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 settings.

@Conaclos Conaclos changed the title feat(rome_js_analyze): noDuplicateClassMember feat(rome_js_analyze): noDuplicateClassMembers Jan 1, 2023
@ematipico
Copy link
Contributor

ematipico commented Jan 1, 2023

@Conaclos The implementation of this rule was already assigned to someone else: #3963

@Conaclos Conaclos closed this Jan 1, 2023
@Conaclos Conaclos deleted the no_duplicate_class_members branch March 7, 2023 16:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

noDuplicateClassMembers, no-dupe-class-members
2 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载