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

Remove deprecated syntax option #6414

Closed
ybiquitous opened this issue Oct 17, 2022 · 2 comments · Fixed by #6420
Closed

Remove deprecated syntax option #6414

ybiquitous opened this issue Oct 17, 2022 · 2 comments · Fixed by #6420
Labels
status: ready to implement is ready to be worked on by someone type: enhancement a new feature that isn't related to rules
Milestone

Comments

@ybiquitous
Copy link
Member

When releasing v14.0.0, we deprecated the syntax option to require using the customSyntax option instead.

### `syntax` option and automatic inferral of syntax

With the next major version, we can remove the deprecated option. See below:

/** @deprecated Use `customSyntax` instead. Using this option will result in an error. */
syntax?: string;

@ybiquitous ybiquitous added status: ready to implement is ready to be worked on by someone type: enhancement a new feature that isn't related to rules labels Oct 17, 2022
@ybiquitous ybiquitous added this to the future-major milestone Oct 17, 2022
@fpetrakov
Copy link
Contributor

@ybiquitous I guess we can close this issue?

@ybiquitous
Copy link
Member Author

@fpetrakov Ah, thanks. I forgot that auto-closing doesn't work on the non-main branches. 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready to implement is ready to be worked on by someone type: enhancement a new feature that isn't related to rules
Development

Successfully merging a pull request may close this issue.

2 participants