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

[Extraction] prep for go.mod & go.work FilePath ReplaceDirective work #1780

Conversation

stefanpenner
Copy link
Contributor

@stefanpenner stefanpenner commented Apr 10, 2024

What type of PR is this?

Extraction as per #1776 (comment)

What package or component does this PR mostly affect?

cmd/fetch_repo

What does this PR do? Why is it needed?

Extraction to simplify #1776 and separated as it's own PR as per #1776 (comment)

Existing tests should cover this sufficiently.

Which issues(s) does this PR fix?

N/A

Other notes for review

@stefanpenner stefanpenner force-pushed the go-refactorings-for-file-path-replace-work branch 7 times, most recently from e86b131 to d87bdae Compare April 10, 2024 20:43
@stefanpenner stefanpenner changed the title [Refactoring] prep for go.mod & go.work FilePath ReplaceDirective work [Extraction] prep for go.mod & go.work FilePath ReplaceDirective work Apr 10, 2024
@stefanpenner stefanpenner force-pushed the go-refactorings-for-file-path-replace-work branch 2 times, most recently from 9ff1a3c to 8e62680 Compare April 10, 2024 20:51
@stefanpenner stefanpenner marked this pull request as ready for review April 10, 2024 20:54
cmd/fetch_repo/go_mod_download.go Outdated Show resolved Hide resolved
@stefanpenner stefanpenner force-pushed the go-refactorings-for-file-path-replace-work branch from 8e62680 to d10e934 Compare April 11, 2024 15:49
@stefanpenner stefanpenner force-pushed the go-refactorings-for-file-path-replace-work branch from d10e934 to d4627be Compare April 11, 2024 15:50
@stefanpenner
Copy link
Contributor Author

@fmeum updated based on your feedback & rebased.

@stefanpenner stefanpenner requested a review from fmeum April 11, 2024 15:50
@fmeum fmeum enabled auto-merge (squash) April 11, 2024 15:52
@fmeum fmeum merged commit 26bb2ac into bazelbuild:master Apr 11, 2024
15 checks passed
@stefanpenner stefanpenner deleted the go-refactorings-for-file-path-replace-work branch April 12, 2024 16:33
@stefanpenner stefanpenner restored the go-refactorings-for-file-path-replace-work branch April 12, 2024 16:36
@stefanpenner stefanpenner deleted the go-refactorings-for-file-path-replace-work branch April 30, 2024 01:20
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.

None yet

2 participants