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

improve error handling of developer workflow tool #5952

Merged
merged 2 commits into from Sep 14, 2023

Conversation

sokra
Copy link
Member

@sokra sokra commented Sep 14, 2023

Description

improve error handling in the developer workflow tooling

Closes WEB-1563

@vercel
Copy link

vercel bot commented Sep 14, 2023

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

Name Status Preview Comments Updated (UTC)
examples-basic-web 🔄 Building (Inspect) Visit Preview 💬 Add feedback Sep 14, 2023 1:55pm
examples-kitchensink-blog 🔄 Building (Inspect) Visit Preview 💬 Add feedback Sep 14, 2023 1:55pm
examples-native-web 🔄 Building (Inspect) Visit Preview 💬 Add feedback Sep 14, 2023 1:55pm
8 Ignored Deployments
Name Status Preview Comments Updated (UTC)
examples-cra-web ⬜️ Ignored (Inspect) Visit Preview Sep 14, 2023 1:55pm
examples-designsystem-docs ⬜️ Ignored (Inspect) Visit Preview Sep 14, 2023 1:55pm
examples-gatsby-web ⬜️ Ignored (Inspect) Visit Preview Sep 14, 2023 1:55pm
examples-nonmonorepo ⬜️ Ignored (Inspect) Sep 14, 2023 1:55pm
examples-svelte-web ⬜️ Ignored (Inspect) Visit Preview Sep 14, 2023 1:55pm
examples-tailwind-web ⬜️ Ignored (Inspect) Visit Preview Sep 14, 2023 1:55pm
examples-vite-web ⬜️ Ignored (Inspect) Sep 14, 2023 1:55pm
turbo-site ⬜️ Ignored (Inspect) Sep 14, 2023 1:55pm

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

@github-actions
Copy link
Contributor

github-actions bot commented Sep 14, 2023

🟢 CI successful 🟢

Thanks

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

@sokra sokra changed the title improve error handling improve error handling of developer workflow tool Sep 14, 2023
throw new Error(
`Measurement ${relativeTo} is not a "${unit}" measurement`
);
}
reportedValue -= prev.value;
}
currentScenario.iface.measurement(
await currentScenario.iface.measurement(
Copy link
Member Author

Choose a reason for hiding this comment

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

measurement is actually async and to handle errors correctly we need to await it

@sokra sokra merged commit f2c639d into main Sep 14, 2023
49 checks passed
@sokra sokra deleted the sokra/devlow-error-handling branch September 14, 2023 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants