Skip to content

Remove duplicated CentOS 7 mirrorlist update logic#1586

Merged
abdurriq merged 2 commits intodevcontainers:mainfrom
7006:patch-1
Mar 5, 2026
Merged

Remove duplicated CentOS 7 mirrorlist update logic#1586
abdurriq merged 2 commits intodevcontainers:mainfrom
7006:patch-1

Conversation

@7006
Copy link
Contributor

@7006 7006 commented Feb 21, 2026

Remove duplicated CentOS 7 mirrorlist handling introduced in this commit 52c79b4

@7006 7006 requested a review from a team as a code owner February 21, 2026 22:32
Copy link
Contributor

@Kaniska244 Kaniska244 left a comment

Choose a reason for hiding this comment

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

Hello @7006

Thank you for the contribution. Would you kindly bump the common-utils feature version?

7006 added 2 commits March 2, 2026 00:20
Remove duplicated CentOS 7 mirrorlist handling introduced in this commit devcontainers@52c79b4
@7006
Copy link
Contributor Author

7006 commented Mar 1, 2026

@microsoft-github-policy-service agree

@7006 7006 requested a review from Kaniska244 March 1, 2026 22:28
@7006
Copy link
Contributor Author

7006 commented Mar 1, 2026

Hello @7006

Thank you for the contribution. Would you kindly bump the common-utils feature version?

@Kaniska244 done

Copy link
Contributor

@Kaniska244 Kaniska244 left a comment

Choose a reason for hiding this comment

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

Looks fine. To be further reviewed by maintainers

@Kaniska244 Kaniska244 requested a review from abdurriq March 2, 2026 09:46
Copy link
Contributor

@abdurriq abdurriq left a comment

Choose a reason for hiding this comment

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

Why should this be removed? From the commit linked in the description, it seems that it is needed in this feature to.

@abdurriq
Copy link
Contributor

abdurriq commented Mar 5, 2026

Ah, I see that the block is duplicated within the file. Approved.

@abdurriq abdurriq merged commit 0c2cd3f into devcontainers:main Mar 5, 2026
12 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