Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add prefer-arrow-function rule #19

Merged
merged 3 commits into from
Mar 2, 2025

Conversation

arkadiuszbachorski
Copy link
Collaborator

Add prefer-arrow-function rule

♻️ Current situation & Problem

There are arrow and plain functions in JavaScript. To keep consistency across the codes, we want to enforce usage of one type. This is opinionated, but I believe arrow functions are cleaner.

⚙️ Release Notes

  • Add prefer-arrow-function rule

✅ Testing

I tested it against RadGPT codebase locally.

📝 Code of Conduct & Contributing Guidelines

By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

Copy link
Member

@PSchmiedmayer PSchmiedmayer left a comment

Choose a reason for hiding this comment

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

Sounds good to me

@arkadiuszbachorski arkadiuszbachorski merged commit b7f3616 into main Mar 2, 2025
3 checks passed
@arkadiuszbachorski arkadiuszbachorski deleted the arek/add-prefer-arrow-plugin branch March 2, 2025 02:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants