Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

gitserver: Move removal of stray tmp pack files to janitor#62614

Merged
eseliger merged 1 commit into
mainfrom
es/05-13-gitservermoveremovalofstraytmppackfilestojanitor
May 13, 2024
Merged

gitserver: Move removal of stray tmp pack files to janitor#62614
eseliger merged 1 commit into
mainfrom
es/05-13-gitservermoveremovalofstraytmppackfilestojanitor

Conversation

@eseliger

@eseliger eseliger commented May 13, 2024

Copy link
Copy Markdown
Member

There was an open TODO comment to move this to janitor, so that's what this PR does.

Test plan:

Integration and E2E tests are still passing.

@cla-bot cla-bot Bot added the cla-signed label May 13, 2024
This was referenced May 13, 2024
Merged
Merged
Merged
Merged
@eseliger Graphite App

eseliger commented May 13, 2024 •
edited
Loading

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @eseliger and the rest of your teammates on Graphite Graphite

@eseliger eseliger mentioned this pull request May 13, 2024
Merged
@github-actions github-actions Bot added team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all labels May 13, 2024
This was referenced May 13, 2024
Merged
Closed
@eseliger eseliger force-pushed the es/05-13-gitserverusethesamegitbackendsourceforallcodepaths branch from 1d73cf2 to fc236cf Compare May 13, 2024 03:13
@eseliger eseliger force-pushed the es/05-13-gitservermoveremovalofstraytmppackfilestojanitor branch from bf8ed84 to 88e9cca Compare May 13, 2024 03:13
@eseliger eseliger marked this pull request as ready for review May 13, 2024 03:31
@eseliger eseliger requested a review from a team May 13, 2024 03:31
@eseliger eseliger force-pushed the es/05-13-gitserverusethesamegitbackendsourceforallcodepaths branch from fc236cf to 5dc1957 Compare May 13, 2024 13:15
@eseliger eseliger force-pushed the es/05-13-gitservermoveremovalofstraytmppackfilestojanitor branch from 88e9cca to dbc60f7 Compare May 13, 2024 13:15
@eseliger eseliger force-pushed the es/05-13-gitserverusethesamegitbackendsourceforallcodepaths branch from 5dc1957 to 866d71e Compare May 13, 2024 14:07
Base automatically changed from es/05-13-gitserverusethesamegitbackendsourceforallcodepaths to main May 13, 2024 14:15
@eseliger
gitserver: Move removal of stray tmp pack files to janitor
ce5274c 
There was an open TODO comment to move this to janitor, so that's what this PR does.

Test plan:

Integration and E2E tests are still passing.
@eseliger eseliger force-pushed the es/05-13-gitservermoveremovalofstraytmppackfilestojanitor branch from dbc60f7 to ce5274c Compare May 13, 2024 14:35
@eseliger Graphite App

eseliger commented May 13, 2024 •
edited
Loading

Copy link
Copy Markdown
Member Author

Merge activity

  • May 13, 10:35 AM EDT: Graphite rebased this pull request as part of a merge.
  • May 13, 10:44 AM EDT: @eseliger merged this pull request with Graphite.

@eseliger eseliger merged commit 1c93c6e into main May 13, 2024
@eseliger eseliger deleted the es/05-13-gitservermoveremovalofstraytmppackfilestojanitor branch May 13, 2024 14:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@pjlast pjlast pjlast approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla-signed team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@eseliger @pjlast