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

Rename eShopLite to TestShop #2095

Merged
merged 4 commits into from
Feb 7, 2024
Merged

Conversation

eerhardt
Copy link
Member

@eerhardt eerhardt commented Feb 6, 2024

Microsoft Reviewers: Open in CodeFlow

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-integrations Issues pertaining to Aspire Integrations packages label Feb 6, 2024
@danmoseley
Copy link
Member

Is anyone going to read AspireShop and think it might represent something they should look at and copy from? That's why I was thinking of a more random name.
But whatever..

@JamesNK
Copy link
Member

JamesNK commented Feb 6, 2024

✨ TestShop ✨

@eerhardt
Copy link
Member Author

eerhardt commented Feb 6, 2024

✨ BikeShedShop✨

@eerhardt eerhardt changed the title Rename eShopLite to AspireShop Rename eShopLite to TestShop Feb 6, 2024
docs/contributing.md Outdated Show resolved Hide resolved
docs/contributing.md Outdated Show resolved Hide resolved
@eerhardt
Copy link
Member Author

eerhardt commented Feb 6, 2024

Any thoughts here? I went with TestShop, thanks to @JamesNK suggestion.

@danmoseley
Copy link
Member

other than Violet 😄 ? testshop seems fine to me

@eerhardt eerhardt merged commit 86cd6fe into dotnet:main Feb 7, 2024
8 checks passed
@eerhardt eerhardt deleted the RenameeShopLite branch February 7, 2024 00:29
radical pushed a commit to radical/aspire that referenced this pull request Feb 7, 2024
* Rename eShopLite to TestShop
@github-actions github-actions bot locked and limited conversation to collaborators Apr 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-integrations Issues pertaining to Aspire Integrations packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants