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

chore: rename recipes catalog to recipe catalog #2087

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

cdrage
Copy link
Contributor

@cdrage cdrage commented Nov 12, 2024

chore: rename recipes catalog to recipe catalog

What does this PR do?

Searching up the grammar, Recipes Catalog is grammatically incorrect
according to multiple sources and should be Recipe Catalog.

Courtesy of llama3 LLM as well as grammarly AI:

"Recipe" acts as an attributive noun, describing the type of catalog, and doesn't need to be plural. "Recipe Catalog" implies a catalog that is specifically for recipes."

Screenshot / video of UI

What issues does this PR fix or reference?

N/A

How to test this PR?

N/A

Signed-off-by: Charlie Drage [email protected]

### What does this PR do?

Searching up the grammar, Recipes Catalog is grammatically incorrect
according to multiple sources and should be Recipe Catalog.

Courtesy of llama3 LLM as well as grammarly AI:

`"Recipe" acts as an attributive noun, describing the
type of catalog, and doesn't need to be plural. "Recipe Catalog" implies
a catalog that is specifically for recipes."`

### Screenshot / video of UI

<!-- If this PR is changing UI, please include
screenshots or screencasts showing the difference -->

### What issues does this PR fix or reference?

<!-- Include any related issues from Podman Desktop
repository (or from another issue tracker). -->

N/A

### How to test this PR?

<!-- Please explain steps to reproduce -->

N/A

Signed-off-by: Charlie Drage <[email protected]>
@cdrage cdrage requested review from benoitf, jeffmaury and a team as code owners November 12, 2024 15:46
@cdrage cdrage requested review from feloy and gastoner November 12, 2024 15:46
Copy link
Contributor

@axel7083 axel7083 left a comment

Choose a reason for hiding this comment

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

Thanks for the fix ahah

@odockal
Copy link
Contributor

odockal commented Nov 14, 2024

@ScrewTSW Fyi

@benoitf
Copy link
Collaborator

benoitf commented Nov 15, 2024

3 approvals, more than the characters changed 🚀

@cdrage cdrage merged commit 2cd5ecc into containers:main Nov 15, 2024
6 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.

5 participants