EmmanuelDemey / EmmanuelDemey/eslint-plugin-angular

Better deprecation warning for editor integration

Open
#546 1 comment 0 reactions 0 assignees View on GitHub
Hacktoberfest Improvement
Dominant language
JavaScript
Stars
620
Forks
127
PR merge metrics
No merged PRs in 30d

Description

Hi!

Here

https://github.com/Gillespie59/eslint-plugin-angular/blob/cd58df172c1561994ce980bc9d9042f7a4b11c15/rules/service-name.js#L72

you just `console.warn` which lands on stderr. For SublimeLinter we parse stderr for common errors. It would be way easier to parse this message if you reword this. E.g. `DeprecationWarning: ....`.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in rules/service-name.js at line 72, where the deprecation message is sent to stderr, and read the surrounding rule logic. Check how SublimeLinter parses stderr, then verify that the warning has an explicit deprecation prefix and remains understandable to users.

Written by the indexing model from the issue text.

Assessment

Tech stack
eslint, javascript
Domain
developer-experience, tooling
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.