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.

Sign up for GitHub

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

Jump to bottom

chore: remove the need for an active execution route at all times #391

Merged
merged 5 commits into from
Dec 18, 2024
Merged

chore: remove the need for an active execution route at all times #391

merged 5 commits into from
Dec 18, 2024

Conversation

frontendphil
Copy link
Member

Fixes #389

@vercel Vercel
Copy link

vercel bot commented Dec 18, 2024 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
zodiac-pilot-example-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 18, 2024 2:55pm

@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Dec 18, 2024 •
edited
Loading

Deploying zodiac-pilot with  Cloudflare Pages  Cloudflare Pages

Latest commit: 832e3d4
Status: ✅  Deploy successful!
Preview URL: https://472b8fbe.zodiac-pilot-29m.pages.dev
Branch Preview URL: https://chore-remove-need-for-active.zodiac-pilot-29m.pages.dev

View logs

@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Dec 18, 2024 •
edited
Loading

Deploying zodiac-pilot-connect with  Cloudflare Pages  Cloudflare Pages

Latest commit: 832e3d4
Status: ✅  Deploy successful!
Preview URL: https://5420504d.zodiac-pilot.pages.dev
Branch Preview URL: https://chore-remove-need-for-active.zodiac-pilot.pages.dev

View logs

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 18, 2024 •
edited
Loading

Coverage Report for extension

Status Category Percentage Covered / Total
🔵 Lines 63.55% 5390 / 8481
🔵 Statements 63.55% 5390 / 8481
🔵 Functions 70.23% 328 / 467
🔵 Branches 80.36% 839 / 1044
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
extension/src/panel/pages/routes/index.tsx 0% 0% 0% 0% 1-12
extension/src/panel/pages/routes/list/ListRoutes.tsx 100% 100% 100% 100%
extension/src/panel/pages/routes/list/Route.tsx 97.05% 81.81% 66.66% 97.05% 49, 59
Generated in workflow #235 for commit 832e3d4 by the Vitest Coverage Report Action

@frontendphil frontendphil force-pushed the chore/remove-need-for-active-route branch from d1d2fba to a2ce3c5 Compare December 18, 2024 14:52
@frontendphil frontendphil force-pushed the chore/remove-need-for-active-route branch from 5bdcd03 to 832e3d4 Compare December 18, 2024 14:54
@frontendphil frontendphil marked this pull request as ready for review December 18, 2024 14:54
@frontendphil frontendphil enabled auto-merge (squash) December 18, 2024 14:55
@frontendphil frontendphil merged commit 7615eea into main Dec 18, 2024
12 checks passed
@frontendphil frontendphil deleted the chore/remove-need-for-active-route branch December 18, 2024 14:57
@github-actions github-actions bot locked and limited conversation to collaborators Dec 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove the need for an active route at all times
1 participant
@frontendphil