Switch to dschmidt/easygettext to use typescript-estree #6833
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.
…from typescript-eslint project instead of outdated buntis parser
Description
buntis is outdated and fails to parse newer javascript/typescript syntax like optional chaining.
There is a PR open for porting to the properly maintained typescript-eslint/typescript-estree library, but it hasn't seen any feedback. It does not fix linenumbers anyhow.
I've sent a new PR which also fixes line numbers, I'm not optimistic it will be merged instead ... we should consider creating a fork in owncloud organization. Let's discuss tomorrow...
Related Issue
Motivation and Context
It fixes missing line numbers and parse errors in ts files/vue files using typescript.
How Has This Been Tested?
removed .pot files from .gitignore
used old version to create those files, commited them. used new version and looked at the diff: only a few line numbers diverged slightly, most probably nothing to worry about.
Types of changes
Open tasks: