Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: move sharing to new cloud #3572

Merged
merged 4 commits into from
Apr 2, 2025
Merged

chore: move sharing to new cloud #3572

merged 4 commits into from
Apr 2, 2025

Conversation

sklein12
Copy link
Contributor

@sklein12 sklein12 commented Apr 1, 2025

No description provided.

Copy link
Contributor

gru-agent bot commented Apr 1, 2025

TestGru Assignment

Summary

Link CommitId Status Reason
Detail 073d498 ✅ Finished

Files

File Pull Request
src/commands/eval.ts ❌ Failure (I failed to write the unit tests for the file.)
src/share.ts 🔴 Closed #3574

Tip

You can @gru-agent and leave your feedback. TestGru will make adjustments based on your input

Copy link
Contributor

gru-agent bot commented Apr 1, 2025

TestGru Assignment

Summary

Link CommitId Status Reason
Detail 073d498 🚫 Skipped No files need to be tested {"site/docs/cloud/index.md":"target file(site/docs/cloud/index.md) not in work scope \n include: src/app/src//*.ts \n exclude: node_modules,/.test.ts,**/.test.tsx,/*.spec.ts,/.spec.tsx,**/.d.ts,/*.test.js,/.spec.js","src/app/src/pages/eval/components/ShareModal.tsx":"no test value","src/commands/eval.ts":"no test value","src/commands/share.ts":"target file(src/commands/share.ts) not in work scope \n include: src/app/src/**/.ts \n exclude: node_modules,/*.test.ts,/.test.tsx,**/.spec.ts,/*.spec.tsx,/.d.ts,**/.test.js,/*.spec.js","src/constants.ts":"no test value","src/server/server.ts":"no test value","src/share.ts":"target file(src/share.ts) not in work scope \n include: src/app/src//.ts \n exclude: node_modules,**/.test.ts,/*.test.tsx,/.spec.ts,**/.spec.tsx,/*.d.ts,/.test.js,**/.spec.js","test/share.test.ts":"target file(test/share.test.ts) not in work scope \n include: src/app/src/**/*.ts \n exclude: node_modul…

Tip

You can @gru-agent and leave your feedback. TestGru will make adjustments based on your input

Copy link
Member

@mldangelo mldangelo left a comment

Choose a reason for hiding this comment

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

This is very exciting

@sklein12 sklein12 force-pushed the steve/04012025-moveshare branch from 5eb0870 to b451609 Compare April 1, 2025 21:25
@sklein12 sklein12 force-pushed the steve/04012025-moveshare branch from 0969f38 to 368b56c Compare April 1, 2025 21:37
@sklein12 sklein12 enabled auto-merge (squash) April 2, 2025 18:33
@sklein12 sklein12 disabled auto-merge April 2, 2025 18:44
@sklein12 sklein12 merged commit 612279b into main Apr 2, 2025
27 checks passed
@sklein12 sklein12 deleted the steve/04012025-moveshare branch April 2, 2025 18:44
mldangelo pushed a commit that referenced this pull request Apr 4, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants