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

chore: move snapshot to snapshot relay #2889

Merged
merged 1 commit into from
May 19, 2023
Merged

chore: move snapshot to snapshot relay #2889

merged 1 commit into from
May 19, 2023

Conversation

bigint
Copy link
Member

@bigint bigint commented May 19, 2023

What does this PR do?

🤖 Generated by Copilot at 89ed89a

This pull request moves the snapshot package to the workers/snapshot-relay subdirectory and updates the pnpm-lock.yaml file to use the latest @lenster packages. It also removes some unnecessary files from the snapshot package.

Related issues

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (non-breaking small changes to existing functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Explanation of the changes

🤖 Generated by Copilot at 89ed89a

  • Rename and move Snapshot.graphql and Space.graphql files from packages/snapshot/documents/queries to packages/workers/snapshot-relay/src/documents/queries to reflect the new structure of the snapshot-relay worker (link, link)
  • Delete packages/snapshot folder and its contents as it is no longer needed for the snapshot-relay worker (link, link, link)
  • Add dev property to @lenster/lenster package in pnpm-lock.yaml to indicate that it is only used for development purposes (link)
  • Update tslib dependency from 2.4.1 to 2.5.1 for @lenster/lenster, @lenster/lenster-core, and @peculiar/webcrypto packages in pnpm-lock.yaml to use the latest helper functions for TypeScript code (link, link, link)

Emoji

📦🔧🆙

@vercel
Copy link

vercel bot commented May 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
prerender ✅ Ready (Inspect) Visit Preview May 19, 2023 2:06pm
ui ✅ Ready (Inspect) Visit Preview May 19, 2023 2:06pm
web ✅ Ready (Inspect) Visit Preview May 19, 2023 2:06pm

@vercel vercel bot temporarily deployed to Preview – ui May 19, 2023 14:02 Inactive
@bigint bigint enabled auto-merge (squash) May 19, 2023 14:03
@socket-security
Copy link

New dependency changes detected. Learn more about Socket for GitHub ↗︎


🚨 Potential security issues found in this pull request. To accept the risk, merge this PR and you will not be notified again.

Bot Commands

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of package-name@version specifiers. e.g. @SocketSecurity ignore foo@1.0.0 bar@* or ignore all packages with @SocketSecurity ignore-all

  • @SocketSecurity ignore ui@0.0.0
🧐 Potential typo squat

Package name is similar to other popular packages and may not be the package you want.

Use care when consuming similarly named packages and ensure that you did not intend to consume a different package. Malicious packages often publish using similar names as existing popular packages.

Package 📎 Did you mean? Found in
ui@0.0.0 (added) udi (480 times more downloads) pnpm-lock.yaml, apps/storybook/package.json, pnpm-lock.yaml, apps/web/package.json, pnpm-lock.yaml
Pull request alert summary
Issue Status
Install scripts ✅ 0 issues
Native code ✅ 0 issues
Bin script shell injection ✅ 0 issues
Unresolved require ✅ 0 issues
Invalid package.json ✅ 0 issues
HTTP dependency ✅ 0 issues
Git dependency ✅ 0 issues
Potential typo squat ⚠️ 1 issue
Known Malware ✅ 0 issues
Telemetry ✅ 0 issues
Protestware/Troll package ✅ 0 issues

📊 Modified Dependency Overview:

🚮 Removed packages: snapshot@0.0.0

@vercel vercel bot temporarily deployed to Preview – prerender May 19, 2023 14:03 Inactive
@bigint bigint merged commit 9e414df into main May 19, 2023
5 of 6 checks passed
@bigint bigint deleted the rm-snapshot branch May 19, 2023 14:11
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.

None yet

1 participant