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

perf: Exclude network while doing getReport #9351

Merged
merged 1 commit into from
Nov 21, 2024
Merged

perf: Exclude network while doing getReport #9351

merged 1 commit into from
Nov 21, 2024

Conversation

Tofandel
Copy link
Contributor

@Tofandel Tofandel commented Oct 30, 2024 •
edited
Loading

This will speed up the report generation and avoid unwanted DNS queries during the report

See lovell/detect-libc#21

@Tofandel Tofandel requested a review from a team as a code owner October 30, 2024 14:59
@turbo-orchestrator turbo-orchestrator bot added needs: triage New issues get this label. Remove it after triage owned-by: turborepo pkg: turbo-repository labels Oct 30, 2024
@vercel Vercel
Copy link

vercel bot commented Oct 30, 2024 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
examples-nonmonorepo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 21, 2024 6:27pm
8 Skipped Deployments
Name Status Preview Comments Updated (UTC)
examples-basic-web ⬜️ Ignored (Inspect) Visit Preview Nov 21, 2024 6:27pm
examples-designsystem-docs ⬜️ Ignored (Inspect) Visit Preview Nov 21, 2024 6:27pm
examples-gatsby-web ⬜️ Ignored (Inspect) Visit Preview Nov 21, 2024 6:27pm
examples-kitchensink-blog ⬜️ Ignored (Inspect) Visit Preview Nov 21, 2024 6:27pm
examples-native-web ⬜️ Ignored (Inspect) Visit Preview Nov 21, 2024 6:27pm
examples-svelte-web ⬜️ Ignored (Inspect) Visit Preview Nov 21, 2024 6:27pm
examples-tailwind-web ⬜️ Ignored (Inspect) Visit Preview Nov 21, 2024 6:27pm
examples-vite-web ⬜️ Ignored (Inspect) Visit Preview Nov 21, 2024 6:27pm

@vercel Vercel
Copy link

vercel bot commented Oct 30, 2024

@Tofandel is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@Tofandel Tofandel changed the title Exclude network while doing getReport perf: Exclude network while doing getReport Oct 30, 2024
@ijjk
Copy link
Member

ijjk commented Nov 21, 2024 •
edited by chris-olszewski
Loading

Allow CI Workflow Run

  • approve CI run for commit: f1d8422

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

1 similar comment
@ijjk
Copy link
Member

ijjk commented Nov 21, 2024 •
edited by chris-olszewski
Loading

Allow CI Workflow Run

  • approve CI run for commit: f1d8422

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@chris-olszewski chris-olszewski merged commit b6ddde3 into vercel:main Nov 21, 2024
35 of 38 checks passed
This was referenced Dec 2, 2024
Open
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chris-olszewski chris-olszewski chris-olszewski approved these changes

@dimitropoulos dimitropoulos Awaiting requested review from dimitropoulos dimitropoulos is a code owner automatically assigned from vercel/turbo-oss

Assignees
No one assigned
Labels
needs: triage New issues get this label. Remove it after triage pkg: turbo-repository
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Tofandel @ijjk @chris-olszewski @anthonyshew