Skip to content

Conversation

@petele
Copy link
Contributor

@petele petele commented Aug 8, 2024

There are a handful of deprecated methods that I've included, but think we should probably remove? Looking for confirmation please.

  • compile
  • input
  • lastMatch
  • lastParent
  • leftContext
  • rightContext

@github-actions github-actions bot added the feature definition Creating or defining new features or groups of features. label Aug 8, 2024
- javascript.builtins.RegExp.@@replace
- javascript.builtins.RegExp.@@search
- javascript.builtins.RegExp.@@species
- javascript.builtins.RegExp.@@split
Copy link
Collaborator

Choose a reason for hiding this comment

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

We also need javascript.grammar.regular_expression_literals and all of the non-deprecated javascript.regular_expressions.* keys. It's possible there are some distinct features in there, but it'll be easier to pick them out with the dist file.

@petele
Copy link
Contributor Author

petele commented Aug 13, 2024

Added additional keys, and removed all deprecated functions.

@petele petele requested a review from ddbeck August 13, 2024 14:27
Copy link
Collaborator

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

Thank you, Pete!

@ddbeck ddbeck merged commit c420cb9 into web-platform-dx:main Aug 13, 2024
@petele petele deleted the regexp branch August 14, 2024 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature definition Creating or defining new features or groups of features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants