Skip to content

Conversation

@erik-beus
Copy link
Contributor

The return type of listAll was an object, while in fact it returns an array. See the events.json file on npm for reference in version 1.18.6 as well as in the latest version.

@github-actions
Copy link
Contributor

Thanks for the PR!

This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged.

@erik-beus
Copy link
Contributor Author

@microsoft-github-policy-service agree

Copy link
Contributor

@saschanaz saschanaz left a comment

Choose a reason for hiding this comment

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

Thanks!

@saschanaz
Copy link
Contributor

LGTM

@github-actions
Copy link
Contributor

There was an issue merging, maybe try again saschanaz. Details

@saschanaz
Copy link
Contributor

Error: HttpError: New changes require approval from someone other than the last pusher.

So that certainly does not include me

@jakebailey jakebailey merged commit 17a020b into microsoft:main Nov 23, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants