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 sequelize #2720

Merged
merged 1 commit into from
Sep 11, 2023
Merged

Update sequelize #2720

merged 1 commit into from
Sep 11, 2023

Conversation

Pezmc
Copy link
Contributor

@Pezmc Pezmc commented Sep 8, 2023

As part of updating moment-timezone above 0.5.34

@codecov
Copy link

codecov bot commented Sep 8, 2023

Codecov Report

Merging #2720 (9bea98c) into main (ab1ab07) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2720   +/-   ##
=======================================
  Coverage   39.52%   39.52%           
=======================================
  Files         540      540           
  Lines       19130    19130           
  Branches     4523     4523           
=======================================
  Hits         7562     7562           
  Misses      11568    11568           
Flag Coverage Δ
backend 74.45% <ø> (ø)
frontend 2.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@Pezmc Pezmc requested a review from knolleary September 8, 2023 21:51
@Steve-Mcl
Copy link
Contributor

bah - conflicts @Pezmc

@Pezmc
Copy link
Contributor Author

Pezmc commented Sep 11, 2023

On it

@Pezmc
Copy link
Contributor Author

Pezmc commented Sep 11, 2023

Rebased on top of main

@Steve-Mcl
Copy link
Contributor

@Pezmc I realise this is the familiar "limit Allocation failed" error, but as this is related to DB stuff, I think we should try to get at leasy 1 pass on PG tests?

@Steve-Mcl Steve-Mcl merged commit 72db2e5 into main Sep 11, 2023
5 checks passed
@Steve-Mcl Steve-Mcl deleted the chore-update-sequelize branch September 11, 2023 08:24
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.

2 participants