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

Update web dependencies #299

Closed
1 of 4 tasks
Tracked by #304
yankovs opened this issue Dec 19, 2022 · 2 comments · Fixed by #358
Closed
1 of 4 tasks
Tracked by #304

Update web dependencies #299

yankovs opened this issue Dec 19, 2022 · 2 comments · Fixed by #358
Milestone

Comments

@yankovs
Copy link
Contributor

yankovs commented Dec 19, 2022

Feature Category

  • Correctness
  • User Interface / User Experience
  • Performance
  • Other (please explain)

Describe the problem

Packages are old, this causes our CI/CD deployment of mquery (web) to indicate a lot of warnings:
image

Describe the solution you'd like

Upgrade these, check it works, move on and be happy with your life

Describe alternatives you've considered

@msm-code
Copy link
Contributor

Updating is easy, checking if it works and fixing the regressions is not. Last time we did this (#260) we later had two regressions (#263, #294) even though it was tested. And bumping is not a very high priority, since security surface is very low (most vulnerabilities reported by CI aren't)

But of course agreed, we should bump the dependencies (and ideally, keep them updated in the future).

@msm-code msm-code mentioned this issue Dec 21, 2022
6 tasks
@msm-code msm-code added this to the v1.4.0 milestone Dec 27, 2022
@msm-code
Copy link
Contributor

Just head's up: I plan to work on that, but recently mquery's development picked up, and I don't want to work on this when there are PRs pending (this will probably cause merge conflicts).

@yankovs yankovs mentioned this issue Mar 16, 2023
4 tasks
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 a pull request may close this issue.

2 participants