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

[TS] Add property versions of raw fields to TS #4574

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

Phlosioneer
Copy link
Contributor

JS properties exist for the _type and _input fields, but they aren't present in the typescript definition file. This commit adds those properties.

The commit adds doc comments pointing developers toward the corresponding properties. I don't know if there's a deprication process for removing the old fields. inputStream's setter function does a reset of the lexer, but otherwise the properties are normal.

JS properties exist for the `_type` and `_input` fields, but they aren't
present in the typescript definition file. This commit adds those properties.

The commit adds doc comments pointing developers toward the corresponding
properties. I don't know if there's a deprication process for removing the
old fields. `inputStream`'s setter function does a reset of the lexer, but
otherwise the properties are normal.

Signed-off-by: Phlosioneer <mattmdrr2@gmail.com>
@@ -7,15 +7,23 @@ export declare class Lexer extends Recognizer<number> {

static DEFAULT_MODE: number;

/**
Copy link
Contributor

@ericvergnaud ericvergnaud Apr 6, 2024

Choose a reason for hiding this comment

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

Please use

/**
 * @deprecated .../...
 */

Copy link
Contributor

Choose a reason for hiding this comment

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

@Phlosioneer Hi there, it's been a while... any progress ?

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 this pull request may close these issues.

2 participants