Skip to content

Comment sanitization fix + extension build upgrades #44

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 10 commits into from
Oct 31, 2021
Merged

Conversation

admvx
Copy link
Owner

@admvx admvx commented Oct 31, 2021

No description provided.

admvx and others added 8 commits October 25, 2021 18:54
- Add block comment mid statement in `ParseTest.as`
- Update Jest snapshots
- Add new tests to check for parsing bug (tests currently fail)

Refs #36
- Exclude coverage checks from primary test script
- Add separate test:coverage task
- Add new test:resnap task to update jest snapshots
- Use npm `--prefix` option in test tasks instead of changing directory
- Clear defunct and redundant options from base config
- Add new dual config to gather client + server configs in array
- Update primary build task to use dual config
- Rename primary build task to `compile` (previously `webpack`)

Refs #43
- Add `watch` task (back) for local development
- Use `-- prefix` npm option in place of `cd` for `postinstall` task
- Remove `compile` task (not working with webpack and rarely needed)
- Refactor `watch` to work with webpack and add problem matcher
- Add `Terminate watch` task to tidy up after running launch profile
- Use vscode `Watch` task in place of direct `npm:watch`
- Call `Terminate watch` at end of debug session to clear orphaned task
- Set 30s timeout on `Attach to Server` to cover watch launch delay

Refs #43
Recycling character index references caused problems when the index was incremented more than once per cycle

Fixes #36
@admvx admvx merged commit ecd842e into main Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Breakpoints no longer work when debugging the extension inside Visual Studio Code Block comment sanitization strips too many characters
1 participant