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

Fix IngressClasses example #340

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kmadel
Copy link
Contributor

@kmadel kmadel commented Oct 31, 2024

The IngressClasses summary is incorrect and the example is misleading.

The IngressClasses summary is incorrect and the example is misleading.
Copy link

netlify bot commented Oct 31, 2024

Deploy Preview for vcluster-docs-site ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 8101db1
🔍 Latest deploy log https://app.netlify.com/sites/vcluster-docs-site/deploys/672a33bb8310650008754191
😎 Deploy Preview https://deploy-preview-340--vcluster-docs-site.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 configuration.

@Piotr1215
Copy link
Contributor

Could you please allow edits by maintainers, so I can help you with vale?

@kmadel
Copy link
Contributor Author

kmadel commented Nov 4, 2024

Could you please allow edits by maintainers, so I can help you with vale?

I already see that "Allow edits and access to secrets by maintainers" is checked. What else needs to be done to allow edits?

Signed-off-by: Piotr Zaniewski <[email protected]>
@Piotr1215
Copy link
Contributor

Piotr1215 commented Nov 5, 2024

I'm still not able to push my changes:

➜ git push --force-with-lease fork quick-start-explore-ingress-classes-fix
Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
To github.com:kmadel/vcluster-docs.git
 ! [remote rejected]   quick-start-explore-ingress-classes-fix -> quick-start-explore-ingress-classes-fix (permission denied)
error: failed to push some refs to 'github.com:kmadel/vcluster-docs.git'

This worked on another PR, double-check that settings are applied (toggle them maybe), you can also add me as collaborator to your fork.

You can also fix the vale errors yourself, the comments from vale bot are in the files tab. Before you do however, please rebase the PR on main to pull the latest changes.

@kmadel
Copy link
Contributor Author

kmadel commented Nov 5, 2024

I toggled the Allow Edits settings and added you as a contributor to my fork.

deniseschannon
deniseschannon previously approved these changes Nov 5, 2024
@deniseschannon deniseschannon dismissed their stale review November 5, 2024 14:27

Didn't see the vale settings failure

@Piotr1215
Copy link
Contributor

@kmadel it worked this time, I have also ported the document to main docs. If I understand correctly, this change is for the v0.20 as well as the upcoming version?

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