Skip to content
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

V2 #20

Merged
merged 11 commits into from
Nov 24, 2023
Merged

V2 #20

merged 11 commits into from
Nov 24, 2023

Conversation

ryanvade
Copy link
Owner

Duplicate of #15, #8.

closes #19
closes #18
closes #13
closes #12

…atures

- bump action to Node.js 20
- update dependencies
- add optional separator parameter
- add optional keyAnywhereInTitle parameter
- update docs
- add eslint config file

BREAKING CHANGE: Now requires Node.js 20
…atures

- bump action to Node.js 20
- update dependencies
- add optional separator parameter
- add optional keyAnywhereInTitle parameter
- update docs
- add eslint config file

BREAKING CHANGE: Now requires Node.js 20
- move default docs for separator in readme
- reset environment variables on each test
- switch to qodana action 2023.2.8
Copy link

github-actions bot commented Nov 24, 2023

Qodana for JS

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

- fix typo in test workflow
- install node dependencies in qodana workflow
- set environment variables at start of tests
- fix eslint errors in regex
- move most logic to main.ts
- don't need to call run in internal tests
@ryanvade ryanvade merged commit 07488cd into main Nov 24, 2023
3 checks passed
@ryanvade ryanvade deleted the v2 branch November 24, 2023 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant