Skip to content

Conversation

@consideRatio
Copy link
Contributor

@consideRatio consideRatio commented Nov 25, 2025

As part of #7290, the helm chart repo was changed, but the bitnami reference wasn't. This PR fixes that documentation, removes another example references to bitnami, and removes a unused link.

@consideRatio consideRatio requested a review from a team as a code owner November 25, 2025 11:17
@consideRatio consideRatio requested review from sunjayBhatia and tsaarni and removed request for a team November 25, 2025 11:17
@sunjayBhatia sunjayBhatia requested review from a team, clayton-gonsalves and izturn and removed request for a team November 25, 2025 11:18
@github-actions
Copy link

Hi @consideRatio! Welcome to our community and thank you for opening your first Pull Request. Someone will review it soon. Thank you for committing to making Contour better. You can also join us on our mailing list and in our channel in the Kubernetes Slack Workspace

@codecov
Copy link

codecov bot commented Nov 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.85%. Comparing base (24c6a57) to head (23e2396).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #7318   +/-   ##
=======================================
  Coverage   81.85%   81.85%           
=======================================
  Files         130      130           
  Lines       15747    15747           
=======================================
  Hits        12889    12889           
  Misses       2574     2574           
  Partials      284      284           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tsaarni tsaarni added the release-note/none-required Marks a PR as not requiring a release note. Should only be used for very small changes. label Nov 25, 2025
Copy link
Member

@tsaarni tsaarni left a comment

Choose a reason for hiding this comment

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

@consideRatio Good catch, thank you for contributing!

@tsaarni tsaarni enabled auto-merge (squash) November 25, 2025 11:47
@consideRatio
Copy link
Contributor Author

consideRatio commented Nov 25, 2025

Wiee thank you for the instant review @tsaarni! I am also thankful I wasn't asked to create an issue first for this detail fix first!

@tsaarni tsaarni merged commit 180ff16 into projectcontour:main Nov 25, 2025
27 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/none-required Marks a PR as not requiring a release note. Should only be used for very small changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants