Skip to content
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

Add no-deprecated-deno #650

Closed
lucacasonato opened this issue Apr 1, 2021 · 2 comments · Fixed by #651
Closed

Add no-deprecated-deno #650

lucacasonato opened this issue Apr 1, 2021 · 2 comments · Fixed by #651

Comments

@lucacasonato
Copy link
Member

This lint would error on all uses of the APIs that we are going to deprecate as per denoland/deno#9795. The lint only has to appear on the most obvious cases (e.g. new Deno.Buffer, but not const { Buffer } = Deno; new Buffer();).

The lint rule should be recommended.

For Deno 1.9 the lint should highlight the following APIs:

  • Deno.Buffer
  • Deno.readAll
  • Deno.readAllSync
  • Deno.writeAll
  • Deno.writeAllSync

Rule name is open for bikeshedding :-)

@magurotuna
Copy link
Member

How about clarifying a little more by adding api as postfix like no-deprecated-deno-api?

Anyway, I would like to work on it :D

@lucacasonato
Copy link
Member Author

Sounds good. Let's bike-shed the name in the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants