Skip to content

feat: add image property for better organization#4426

Merged
bwateratmsft merged 1 commit intomicrosoft:mainfrom
yoyoys:image-property
Dec 5, 2024
Merged

feat: add image property for better organization#4426
bwateratmsft merged 1 commit intomicrosoft:mainfrom
yoyoys:image-property

Conversation

@yoyoys
Copy link
Contributor

@yoyoys yoyoys commented Dec 3, 2024

I'm using Google Cloud and the repository path is so mess up and useless like this:
old

And I've add RegistryAndPath and RepositoryNameShort and to filter them like this:
better

The code is tested and pass all the built-in testes,
and no effect to current users until they use new image property,
hope this is helpful for everyone. Thanks

@yoyoys yoyoys requested a review from a team as a code owner December 3, 2024 10:06
@yoyoys
Copy link
Contributor Author

yoyoys commented Dec 3, 2024

@microsoft-github-policy-service agree

Copy link
Collaborator

@bwateratmsft bwateratmsft left a comment

Choose a reason for hiding this comment

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

Code looks good to me. @fiveisprime any objections to this from PM perspective?

@fiveisprime
Copy link
Member

No objection from me, looks good. :shipit:

@bwateratmsft bwateratmsft merged commit a45a3df into microsoft:main Dec 5, 2024
@bwateratmsft
Copy link
Collaborator

Thanks for the contribution @yoyoys!

@yoyoys
Copy link
Contributor Author

yoyoys commented Dec 10, 2024

@bwateratmsft Would you add this to milestones? :)

@bwateratmsft bwateratmsft added this to the 1.30.0 milestone Dec 10, 2024
@bwateratmsft
Copy link
Collaborator

Done! Not sure when it will be shipping though, given the holidays.

@bwateratmsft bwateratmsft modified the milestones: 1.30.0, 1.29.4 Jan 9, 2025
@microsoft microsoft locked and limited conversation to collaborators Jan 21, 2025
@bwateratmsft
Copy link
Collaborator

This is now released in Docker extension version 1.29.4.

@yoyoys yoyoys deleted the image-property branch February 17, 2025 02:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants