Skip to content

Swift: cache file labels #9818

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

Merged
merged 3 commits into from
Jul 27, 2022
Merged

Conversation

redsun82
Copy link
Contributor

This required a bit of a generalization of TrapLabelStore to not
work only with pointers.

@github-actions github-actions bot added the Swift label Jul 13, 2022
@redsun82 redsun82 force-pushed the redsun82/swift-file-label-caching branch from 428de13 to b9b90b6 Compare July 14, 2022 04:23
Base automatically changed from redsun82/swift-exclusive-file to main July 14, 2022 14:23
@redsun82 redsun82 force-pushed the redsun82/swift-file-label-caching branch from b9b90b6 to b72e116 Compare July 14, 2022 14:24
This required a bit of a generalization of `TrapLabelStore` to not
work only with pointers.
@redsun82 redsun82 force-pushed the redsun82/swift-file-label-caching branch from b72e116 to 5e74df3 Compare July 14, 2022 14:32
@redsun82 redsun82 marked this pull request as ready for review July 18, 2022 07:35
@redsun82 redsun82 requested a review from a team as a code owner July 18, 2022 07:35
@redsun82 redsun82 changed the title Swift: cache file paths Swift: cache file labels Jul 20, 2022
AlexDenisov
AlexDenisov previously approved these changes Jul 26, 2022
Copy link
Contributor

@AlexDenisov AlexDenisov left a comment

Choose a reason for hiding this comment

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

:shipit:

@AlexDenisov AlexDenisov merged commit f9bdca3 into main Jul 27, 2022
@AlexDenisov AlexDenisov deleted the redsun82/swift-file-label-caching branch July 27, 2022 07:12
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