Skip to content

Conversation

dplewis
Copy link
Member

@dplewis dplewis commented Feb 4, 2023

Pull Request

Issue

I want to see what the effects are against the CI.

Closes: #1728

Approach

  • Update package-lockfile to V2
  • Update lockfile checker in CI
  • Improve CI caching to individual cache per node version
  • Move mongodb runner from CI to Dev Dependencies
  • Update semantic-release packages

@parse-github-assistant
Copy link

parse-github-assistant bot commented Feb 4, 2023

Thanks for opening this pull request!

  • ❌ Please edit your post and use the provided template when creating a new pull request. This helps everyone to understand your post better and asks for essential information to quicker review the pull request.

@codecov
Copy link

codecov bot commented Feb 4, 2023

Codecov Report

Base: 99.89% // Head: 99.89% // No change to project coverage 👍

Coverage data is based on head (3c18815) compared to base (16535a4).
Patch has no changes to coverable lines.

❗ Current head 3c18815 differs from pull request most recent head e35cd62. Consider uploading reports for the commit e35cd62 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##            alpha    #1729   +/-   ##
=======================================
  Coverage   99.89%   99.89%           
=======================================
  Files          61       61           
  Lines        5992     5992           
  Branches     1374     1374           
=======================================
  Hits         5986     5986           
  Misses          6        6           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@dplewis
Copy link
Member Author

dplewis commented Feb 4, 2023

@mtrezza Looks like mongodb-runner and node 14 are working well here.

@dplewis dplewis requested a review from a team February 4, 2023 03:54
@mtrezza
Copy link
Member

mtrezza commented Feb 4, 2023

Looks like mongodb-runner and node 14 are working well here.

Great to hear, that's why we didn't see the issue in the Parse Server CI. So we can keep mongodb-runner. Btw, they made a recent release 3 days ago (albeit just a tag), so there is some maintenance going on it seems. What's left is to move it to the dev dependencies, but we'll do that in a separate PR related to #1730 since we want it to have a distinct changelog entry.

@mtrezza mtrezza changed the title feat: Upgrade Node Package Manager lock file package-lock.json to ver… feat: Upgrade Node Package Manager lock file package-lock.json to version 2 Feb 4, 2023
Copy link
Member

@mtrezza mtrezza left a comment

Choose a reason for hiding this comment

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

Looks good! Get's get this merged if the CI passes...

@mtrezza
Copy link
Member

mtrezza commented Feb 4, 2023

Uh, there is a conflict after merging alpha, could you take a look?

@dplewis dplewis closed this Feb 4, 2023
@dplewis dplewis reopened this Feb 4, 2023
Copy link
Member

@mtrezza mtrezza left a comment

Choose a reason for hiding this comment

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

Looks good!

@mtrezza
Copy link
Member

mtrezza commented Feb 4, 2023

This PR seems to be done, can we merge? Any other issues please in separate PRs.

@dplewis
Copy link
Member Author

dplewis commented Feb 4, 2023

Its good to merge

@mtrezza mtrezza merged commit e993786 into parse-community:alpha Feb 4, 2023
@dplewis dplewis deleted the package-lock-json-v2 branch February 4, 2023 22:28
@mtrezza
Copy link
Member

mtrezza commented Feb 4, 2023

Glad to see this Node 14 issue finally fixed

parseplatformorg pushed a commit that referenced this pull request Feb 4, 2023
# [4.0.0-alpha.10](4.0.0-alpha.9...4.0.0-alpha.10) (2023-02-04)

### Features

* Upgrade Node Package Manager lock file `package-lock.json` to version 2 ([#1729](#1729)) ([e993786](e993786))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 4.0.0-alpha.10

@parseplatformorg parseplatformorg added the state:released-alpha Released as alpha version label Feb 4, 2023
@dplewis dplewis mentioned this pull request Feb 5, 2023
4 tasks
parseplatformorg pushed a commit that referenced this pull request Mar 1, 2023
# [4.1.0-beta.1](4.0.1...4.1.0-beta.1) (2023-03-01)

### Bug Fixes

* `LiveQuerySubscription.unsubscribe` resolves promise before unsubscribing completes ([#1727](#1727)) ([1c96205](1c96205))
* Node engine version upper range is <19 despite Node 19 support ([#1732](#1732)) ([febe187](febe187))
* Saving a new `Parse.Object` with an unsaved `Parse.File` fails ([#1662](#1662)) ([16535a4](16535a4))

### Features

* `LiveQueryClient.close` returns promise when WebSocket closes ([#1735](#1735)) ([979d660](979d660))
* Upgrade Node Package Manager lock file `package-lock.json` to version 2 ([#1729](#1729)) ([e993786](e993786))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 4.1.0-beta.1

@parseplatformorg parseplatformorg added the state:released-beta Released as beta version label Mar 1, 2023
parseplatformorg pushed a commit that referenced this pull request Mar 1, 2023
# [4.1.0-alpha.1](4.0.1...4.1.0-alpha.1) (2023-03-01)

### Bug Fixes

* `LiveQuerySubscription.unsubscribe` resolves promise before unsubscribing completes ([#1727](#1727)) ([1c96205](1c96205))
* Node engine version upper range is <19 despite Node 19 support ([#1732](#1732)) ([febe187](febe187))
* Saving a new `Parse.Object` with an unsaved `Parse.File` fails ([#1662](#1662)) ([16535a4](16535a4))

### Features

* `LiveQueryClient.close` returns promise when WebSocket closes ([#1735](#1735)) ([979d660](979d660))
* Upgrade Node Package Manager lock file `package-lock.json` to version 2 ([#1729](#1729)) ([e993786](e993786))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 4.1.0-alpha.1

parseplatformorg pushed a commit that referenced this pull request May 1, 2023
# [4.1.0](4.0.1...4.1.0) (2023-05-01)

### Bug Fixes

* `LiveQuerySubscription.unsubscribe` resolves promise before unsubscribing completes ([#1727](#1727)) ([1c96205](1c96205))
* Node engine version upper range is <19 despite Node 19 support ([#1732](#1732)) ([febe187](febe187))
* Saving a new `Parse.Object` with an unsaved `Parse.File` fails ([#1662](#1662)) ([16535a4](16535a4))

### Features

* `LiveQueryClient.close` returns promise when WebSocket closes ([#1735](#1735)) ([979d660](979d660))
* Upgrade Node Package Manager lock file `package-lock.json` to version 2 ([#1729](#1729)) ([e993786](e993786))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 4.1.0

@parseplatformorg parseplatformorg added the state:released Released as stable version label May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state:released Released as stable version state:released-alpha Released as alpha version state:released-beta Released as beta version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Regenerate package-lock.json as v2
3 participants