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.

Sign up for GitHub

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

Jump to bottom

Fix API scripts #366

Open

YuryShkoda wants to merge 5 commits into main from cpr-fix

Member

YuryShkoda commented Sep 21, 2023

Issue

cp command is not supported by Windows Command Prompt (cmd.exe).
npm reported vulnerability in jsonwebtoken.

Intent

Improve api scripts to make it work in all shells.
Fix vulnerable dependency.

Implementation

Used cpr package to copy folders within api scripts.
Bumped version of jsonwebtoken.
Regenerated swagger.yaml.

Checks

Code is formatted correctly (npm run lint:fix).
Any new functionality has been unit tested.
All unit tests are passing (npm test).
All CI checks are green.
Reviewer is assigned.

YuryShkoda added 5 commits

September 21, 2023 17:44


          chore(scripts): used cpr package to improve coping

b472f1b


          fix(jsonwebtoken): bumped version to avoid vulnerability


          chore(api): regenerated swagger.yaml

2dc1163


          chore(lint): fixed Session.ts

bf35791


          chore(lint): bumped prettier and fixed lint issues

66232ae

allanbowe approved these changes

View reviewed changes

Member Author

YuryShkoda commented Sep 22, 2023

Unit tests covering api are failing due to #367 issue.

YuryShkoda changed the title ~~Cpr fix~~ Fix API scripts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet