-
Notifications
You must be signed in to change notification settings - Fork 10
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
Update deprecated code #138
Conversation
marcsosduma
commented
Sep 23, 2023
- Migrate from TSLint to ESLint
- Update the deprecated package name 'vscode-debugadapter' and 'debugadapter-testsupport'.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please rebase? It seems that part of the changes are already part of master (which is one of the reasons for the merge conflict which prevents us to directly merge this in).
Other than that: please check the minimal version dependencies, they are very likely much too high.
It represents the closest reasonable ESLint configuration to this | ||
project's original TSLint configuration. | ||
|
||
We recommend eventually switching this configuration to extend from |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you checked that?
Please adjust the comment accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marcsosduma What about this comment?
Good evening, Simon! OK. I will check the points you mentioned. Thank you. |
* Update the deprecated package name 'vscode-debugadapter' and 'debugadapter-testsupport'. Changes: (#134) * Allow viewing of variables with the - character in the name. * Enable hovering on child variables. * Display debug output in the VSCode terminal. * Set XTERM variable to the correct display during debugging with the GDB TTY option. * Utilize gdbtty for debugging on WSL and SSH (Windows -> Linux). * Perform 'stepOver' in subroutines called by 'PERFORM' statement. * Support 'stepInto'/'stepOut' in subroutines called by 'PERFORM' statement. * Run Gdbtty with Ctrl-F5. * Provide support for symbolic link directories on linux. Maybe, just maybe
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please rebase (possibly something along git pull upstream master
, followed by git push --force
) this is still based on an old version
It represents the closest reasonable ESLint configuration to this | ||
project's original TSLint configuration. | ||
|
||
We recommend eventually switching this configuration to extend from |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marcsosduma What about this comment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apart from the log file that should be removed the actual changes are fine.
Please squash the commits to one, then let Oleg merge this in..
LGTM, so let's wait for @OlegKunitsyn to merge that in. |