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

fix(noUnusedVariables): ignore TypeScript property parameters #4214

Merged
merged 1 commit into from
Feb 16, 2023
Merged

fix(noUnusedVariables): ignore TypeScript property parameters #4214

merged 1 commit into from
Feb 16, 2023

Conversation

Conaclos
Copy link
Contributor

Summary

Fixes #4164

Test Plan

New test included.

Documentation

No change.

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

@netlify
Copy link

netlify bot commented Feb 14, 2023

Deploy Preview for docs-rometools canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 2fac94e
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/63ebd67a6d0348000864b808

@Conaclos Conaclos added the A-Linter Area: linter label Feb 15, 2023
@ematipico ematipico added this pull request to the merge queue Feb 16, 2023
Merged via the queue into rome:main with commit 6da1475 Feb 16, 2023
@Conaclos Conaclos deleted the lint/noUnusedVariables/4164 branch March 7, 2023 16:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Linter Area: linter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 lint/correctness/noUnusedVariables quick fix breaks
2 participants