-
Notifications
You must be signed in to change notification settings - Fork 646
chore: add no-top-level-arrow-syntax
rule to Deno Style Guide linter plugin
#6558
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
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6558 +/- ##
=======================================
Coverage 94.57% 94.58%
=======================================
Files 579 579
Lines 43724 43752 +28
Branches 6492 6494 +2
=======================================
+ Hits 41354 41381 +27
- Misses 2330 2331 +1
Partials 40 40 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
no-top-level-arrow-syntax
to Deno Style Guide linter pluginno-top-level-arrow-syntax
rule to Deno Style Guide linter plugin
This is only currently working on canary |
Does this rule include non-exported arrow functions? The example snippet only exemplifies exported arrow functions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Nice!
I think this applies to non exported functions as well. |
Towards #6551
Blocked by denoland/deno#28799