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

fix: cypress dependencies #329

Closed
wants to merge 2 commits into from
Closed

fix: cypress dependencies #329

wants to merge 2 commits into from

Conversation

skjnldsv
Copy link
Contributor

@skjnldsv skjnldsv commented Sep 19, 2023

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv skjnldsv added the 3. to review Waiting for reviews label Sep 19, 2023
@skjnldsv skjnldsv self-assigned this Sep 19, 2023
@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #329 (e070992) into master (be0e999) will decrease coverage by 68.24%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #329       +/-   ##
===========================================
- Coverage   96.47%   28.23%   -68.24%     
===========================================
  Files           5        5               
  Lines          85       85               
  Branches        4        4               
===========================================
- Hits           82       24       -58     
- Misses          3       61       +58     

see 2 files with indirect coverage changes

@artonge
Copy link
Contributor

artonge commented Sep 19, 2023

Why allow 12 and 13 ?

@skjnldsv
Copy link
Contributor Author

Because there was no breaking changes justifying removing 12 in this library :)

@artonge
Copy link
Contributor

artonge commented Sep 20, 2023

Because there was no breaking changes justifying removing 12 in this library :)

From what I understand, 12 triggers some security issues with npm audit

@skjnldsv skjnldsv closed this Sep 20, 2023
@skjnldsv skjnldsv deleted the fix/cypress branch September 20, 2023 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants