Skip to content

#6185 Upgrade avajs to v7#6192

Merged
sosnovsky merged 8 commits intomasterfrom
6185-upgrade-avajs-to-v7
Mar 23, 2026
Merged

#6185 Upgrade avajs to v7#6192
sosnovsky merged 8 commits intomasterfrom
6185-upgrade-avajs-to-v7

Conversation

@ioanatflowcrypt
Copy link
Collaborator

@ioanatflowcrypt ioanatflowcrypt commented Mar 18, 2026

This PR upgraded avajs to v7

close #6185 // if this PR closes an issue


Tests (delete all except exactly one):

  • Does not need tests (refactor only, docs or internal changes)

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

@ioanatflowcrypt
Copy link
Collaborator Author

@sosnovsky Could you tell me what triggers snyk failure?
I don't have acccess to snyk and can't see details.

@sosnovsky
Copy link
Collaborator

Hi @ioanatflowcrypt, error description on Snyk says Failed to parse package.json or package-lock.json as valid JSON, as there are some unresolved merge conflicts in package-lock.json - https://github.com/FlowCrypt/flowcrypt-browser/blob/6185-upgrade-avajs-to-v7/package-lock.json#L747

@ioanatflowcrypt ioanatflowcrypt marked this pull request as ready for review March 21, 2026 12:23
@ioanatflowcrypt
Copy link
Collaborator Author

Thank you. Fixed issue. Please check

sosnovsky
sosnovsky previously approved these changes Mar 23, 2026
Copy link
Collaborator

@sosnovsky sosnovsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well done, works great!

Copy link
Collaborator

@sosnovsky sosnovsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@sosnovsky sosnovsky merged commit 9fe3099 into master Mar 23, 2026
12 checks passed
@sosnovsky sosnovsky deleted the 6185-upgrade-avajs-to-v7 branch March 23, 2026 13:22
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.

Upgrade avajs to v7

2 participants