Skip to content

Conversation

nzaytsev
Copy link
Contributor

@nzaytsev nzaytsev commented Jul 12, 2024

Description

allows store tests subdirectories
lints warnings
ignores trie tests
makes vscode-tests running with vscode/test-cli

Checklist

  • I have followed the guidelines in the Contributing document
  • My changes follow the coding style of this project
  • My changes build without any errors or warnings
  • My changes have been formatted and linted
  • My changes include any required corresponding changes to the documentation (including CHANGELOG.md and README.md)
  • My changes have been rebased and squashed to the minimal number (typically 1) of relevant commits
  • My changes have a descriptive commit message with a short title, including a Fixes $XXX - or Closes #XXX - prefix to auto-close the issue that your PR addresses

@nzaytsev nzaytsev marked this pull request as ready for review July 17, 2024 05:39
@nzaytsev nzaytsev requested review from d13, eamodio and sergeibbb July 18, 2024 08:35
@nzaytsev nzaytsev force-pushed the chore/GLVSC-594_configure_unit_tests branch 3 times, most recently from 72cffdd to aa8d138 Compare July 22, 2024 08:47
@nzaytsev nzaytsev force-pushed the chore/GLVSC-594_configure_unit_tests branch from 57772bb to 739887e Compare July 23, 2024 04:52
@nzaytsev nzaytsev requested a review from d13 July 23, 2024 04:52
@nzaytsev nzaytsev force-pushed the chore/GLVSC-594_configure_unit_tests branch 2 times, most recently from dcddba8 to 9dd106c Compare July 29, 2024 07:01
nzaytsev added 9 commits July 30, 2024 10:56
- improves eslint config
- allows store __tests__ subdirectories
- lints warnings
- Adds more tests for color
- Adds restriction for using .only tests suites
- Adds sample tests for iterable
- Merges webview eslint rules to main confix
- Adds test coverage
@nzaytsev nzaytsev force-pushed the chore/GLVSC-594_configure_unit_tests branch from 9dd106c to 427ce89 Compare July 30, 2024 03:57
@eamodio
Copy link
Member

eamodio commented Aug 19, 2024

Can we split this into an eslint change/upgrade (and take it further to ESLint 9 and the latest typescript-eslint) and then have the unit test work sit on top of that?

@d13 d13 linked an issue Sep 12, 2024 that may be closed by this pull request
@eamodio
Copy link
Member

eamodio commented Sep 18, 2024

Let's get this on top of #3585

@nzaytsev
Copy link
Contributor Author

closed in favor of #3590

@nzaytsev nzaytsev closed this Sep 19, 2024
@nzaytsev nzaytsev removed a link to an issue Sep 19, 2024
@eamodio eamodio deleted the chore/GLVSC-594_configure_unit_tests branch October 14, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants