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 App Repositories to Package Repositories #4795

Merged
merged 8 commits into from
May 31, 2022

Conversation

antgamdia
Copy link
Contributor

@antgamdia antgamdia commented May 27, 2022

Description of the change

Trivial PR just renaming the old "App repo" thing to something more aligned to our concept of "package".
This is a prequel PR, but can be safely merged into main.
~

Benefits

More consistent naming.

Possible drawbacks

N/A

Applicable issues

Additional information

Let's see if the CI agrees :P

EDIT: need to look into the CI fail root cause. Will do soon. It was just a matter of s/has-text("App Repositories")/has-text("Package Repositories")

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
@netlify
Copy link

netlify bot commented May 27, 2022

Deploy Preview for kubeapps-dev ready!

Name Link
🔨 Latest commit 0d4c55d
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/629551e95e21cc000849f971
😎 Deploy Preview https://deploy-preview-4795--kubeapps-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Copy link
Contributor

@absoludity absoludity 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 other drive-by fixes (doc links and wording).

<strong>NOTE:</strong> This App Repository will be created in the "{kubeappsNamespace}"
namespace and packages will be available in all namespaces for installation.
<strong>NOTE:</strong> This Package Repository will be created in the "{kubeappsNamespace}
" namespace and packages will be available in all namespaces for installation.
Copy link
Contributor

Choose a reason for hiding this comment

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

Won't HTML assume that new-line there is a word-break? (Not what's wanted in that situation, with the quote, I don't think?)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

No, IRL it seems it works as expected. It is a breakline introduced by the linter.

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
@antgamdia antgamdia merged commit 8f5261a into vmware-tanzu:main May 31, 2022
@antgamdia antgamdia deleted the pkgRepo-1 branch May 31, 2022 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants