Skip to content

Verify that perf_hooks result actually contains the performance object - #59300

Merged
Jake Bailey (jakebailey) merged 1 commit into
microsoft:mainfrom
jakebailey:fix-59296
Jul 16, 2024
Merged

Verify that perf_hooks result actually contains the performance object#59300
Jake Bailey (jakebailey) merged 1 commit into
microsoft:mainfrom
jakebailey:fix-59296

Conversation

@jakebailey

@jakebailey Jake Bailey (jakebailey) commented Jul 16, 2024

Copy link
Copy Markdown
Member

I guess some environments polyfill this with {} or something. Only use the module if performance is there. This broke in #57875.

Fixes #59296

@jakebailey

Copy link
Copy Markdown
Member Author

TypeScript Bot (@typescript-bot) cherry-pick this to release-5.5

@typescript-bot

TypeScript Bot (typescript-bot) commented Jul 16, 2024

Copy link
Copy Markdown
Contributor

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
cherry-pick this to release-5.5 ✅ Started ✅ Results

@typescript-bot

Copy link
Copy Markdown
Contributor

Hey, Jake Bailey (@jakebailey)! I've created #59301 for you.

@jakebailey
Jake Bailey (jakebailey) merged commit bd54a6b into microsoft:main Jul 16, 2024
@jakebailey
Jake Bailey (jakebailey) deleted the fix-59296 branch July 16, 2024 17:03
Daniel Rosenwasser (DanielRosenwasser) pushed a commit that referenced this pull request Jul 16, 2024
…e-5.5 (#59301)

Co-authored-by: Jake Bailey <5341706+jakebailey@users.noreply.github.com>
@microsoft Microsoft (microsoft) locked as resolved and limited conversation to collaborators Oct 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tryGetPerformanceHooks throws on Stackblitz with ts 5.5.3

4 participants