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

perf: remove unneeded nullish check #745

Merged

Conversation

ericcrosson-bitgo
Copy link
Contributor

In the current implementation, description is always a string, so there
is no point checking if it is not undefined.

This closes
https://github.com/BitGo/api-ts/security/code-scanning/4

In the current implementation, `description` is always a string, so there
is no point checking if it is not undefined.

This closes
https://github.com/BitGo/api-ts/security/code-scanning/4
@ericcrosson-bitgo ericcrosson-bitgo marked this pull request as ready for review April 23, 2024 16:18
@ericcrosson-bitgo ericcrosson-bitgo requested a review from a team as a code owner April 23, 2024 16:18
@bitgopatmcl bitgopatmcl merged commit 978327b into BitGo:master Apr 23, 2024
8 checks passed
Copy link

🎉 This PR is included in version @api-ts/[email protected] 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@ericcrosson-bitgo ericcrosson-bitgo deleted the perf-remove-unneeded-nullish-check branch April 23, 2024 16:23
Copy link

🎉 This PR is included in version @api-ts/[email protected] 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version @api-ts/[email protected] 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version @api-ts/[email protected] 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

Copy link

github-actions bot commented Aug 5, 2024

🎉 This PR is included in version @api-ts/[email protected] 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants