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

Sort by time, rather than the touch function #174623

Merged
merged 1 commit into from
Feb 20, 2023
Merged

Sort by time, rather than the touch function #174623

merged 1 commit into from
Feb 20, 2023

Conversation

RyanCavanaugh
Copy link
Member

It looks extremely likely that this code is using the wrong property. touch is a method that increments the time property.

(Sorry for making a repo-local branch, forgot to push to a fork)

Found by microsoft/TypeScript#52807

@jrieken
Copy link
Member

jrieken commented Feb 20, 2023

That's a great find!

@jrieken jrieken merged commit 2257f33 into main Feb 20, 2023
@jrieken jrieken deleted the fixIdentityService branch February 20, 2023 13:22
@github-actions github-actions bot locked and limited conversation to collaborators Apr 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants