Skip to content

fix: Update RuleVisitor type#135

Merged
fasttime merged 6 commits intomainfrom
rule-visitor-update
Nov 27, 2024
Merged

fix: Update RuleVisitor type#135
fasttime merged 6 commits intomainfrom
rule-visitor-update

Conversation

@nzakas
Copy link
Copy Markdown
Member

@nzakas nzakas commented Nov 25, 2024

Prerequisites checklist

What is the purpose of this pull request?

Make RuleVisitor type work practically with subinterfaces.

What changes did you make? (Give an overview)

  • Converted RuleVisitor interface into a mapped type.

Related Issues

fixes #134

Is there anything you'd like reviewers to focus on?

The typescript-eslint rule recommended using a record instead of an object type. Does that make sense?

@eslint-github-bot eslint-github-bot Bot added the bug Something isn't working label Nov 25, 2024
Comment thread packages/core/src/types.ts Outdated
Co-authored-by: Francesco Trotta <github@fasttime.org>
Comment thread packages/core/src/types.ts Outdated
Co-authored-by: Francesco Trotta <github@fasttime.org>
@fasttime
Copy link
Copy Markdown
Member

Oh, now Prettier 3.4 is complaining because of the formatting of Markdown files. It's the same problem as with eslint/json#67.

image

@nzakas
Copy link
Copy Markdown
Member Author

nzakas commented Nov 27, 2024

I guess their semver policy isn't a strict as ours. 😄

Copy link
Copy Markdown
Member

@fasttime fasttime left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@fasttime fasttime merged commit 156d601 into main Nov 27, 2024
@fasttime fasttime deleted the rule-visitor-update branch November 27, 2024 21:09
@github-actions github-actions Bot mentioned this pull request Nov 27, 2024
snitin315 pushed a commit that referenced this pull request Mar 29, 2026
Co-authored-by: Francesco Trotta <github@fasttime.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted bug Something isn't working

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

Change Request: Update RuleVisitor interface

2 participants