-
Notifications
You must be signed in to change notification settings - Fork 964
Port Regular Expressions checking #3061
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
Merged
Merged
Changes from 26 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
6f4fc7a
Port "Provide Syntax Checking for Regular Expressions"
Andarist a37d964
Port "Improve Recovery of Unterminated Regular Expressions"
Andarist fa64311
Port "Move RegExp flag version mapping to LanguageFeatureMinimumTarget"
Andarist 0499a15
Port "Report RegExp errors in grammar check, use Annex B grammar"
Andarist c091575
Port "Correct Regular Expressions Behavior Related to Annex B"
Andarist f548c2b
Port "Provide Spelling Suggestions for Named Capture Group References…
Andarist f1b8944
Port "Correct regular expression flags scanning for non-BMP characters"
Andarist af449ae
fix stuff
Andarist df6003a
fix stuff
Andarist 1f36b00
fix a thing
Andarist 2593323
fix stuff
Andarist 27c3597
fix stuff
Andarist b9e8ad0
fix stuff
Andarist 0dd18b2
fixup indent
Andarist 8de6161
fix subrange scanning
Andarist a3f863a
add missing case to `scanSourceCharacter`
Andarist b351d4a
fix thing
Andarist db421f4
remove GetNameOfScriptTarget
Andarist f3d4cef
switch it back
Andarist dba4669
bring back comment
Andarist 00ca1e6
restore comment
Andarist 8dd5ac5
fix regExpFlagToFirstAvailableLanguageVersion
Andarist e2b7255
update surrogate handling
Andarist 6998167
accept changes
Andarist a2a0756
accept again ?
Andarist a6e25ad
fix lints
Andarist 9c5ab8d
Merge remote-tracking branch 'origin/main' into port/55600
Andarist 970eeb3
drop the 2015 availability stuff
Andarist ffff27d
remove unused `lastDiagnostic`
Andarist b5b1755
privatize what needs to be privatized
Andarist 66afe1f
reuse `collections.NewSetFromItems`
Andarist ba541a0
Remove unicodeproperties init indirection
Andarist b4ab864
introduce char helper
Andarist 6847b55
passthrough those helpers
Andarist 01b37de
bounds improvement
Andarist 702e8a2
remove redundant comment
Andarist cfcdb99
bring back lost comment
Andarist 8fee12e
add missing asserts
Andarist b2e2b07
handle atoi error
Andarist c65c53d
improve comment
Andarist b4ff2e2
use charAt properly
Andarist 6208841
readd some comments
Andarist 97e002b
tweak
Andarist 96e9c9c
tweak
Andarist File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.