Skip to content

📖 removing duplicated content - #1879

Merged
anik120 merged 1 commit into
operator-framework:mainfrom
grokspawn:catalog-fetching-metas
Mar 26, 2025
Merged

📖 removing duplicated content#1879
anik120 merged 1 commit into
operator-framework:mainfrom
grokspawn:catalog-fetching-metas

Conversation

@grokspawn

@grokspawngrokspawn commented Mar 26, 2025

Copy link
Copy Markdown
Contributor

Description

We have duplicate catalog web service access documentation in a few places:

The howto fetching-catalog-contents.md adds nothing to the knowledgebase, and isn't integrated with other docs, so it represents a good pruning point for duplicate info.

This is the last work item needed to close#1783

Reviewer Checklist

  • API Go Documentation
  • Tests: Unit Tests (and E2E Tests, if appropriate)
  • Comprehensive Commit Messages
  • Links to related GitHub Issue(s)

Signed-off-by: Jordan Keister <jordan@nimblewidget.com>
@grokspawn
grokspawn requested a review from a team as a code ownerMarch 26, 2025 15:40
@netlify

netlifyBot commented Mar 26, 2025

Copy link
Copy Markdown

Deploy Preview for olmv1 ready!

NameLink
🔨 Latest commit428435e
🔍 Latest deploy loghttps://app.netlify.com/sites/olmv1/deploys/67e41ffd399e010008185976
😎 Deploy Previewhttps://deploy-preview-1879--olmv1.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.

@codecov

codecovBot commented Mar 26, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.96%. Comparing base (fc88b93) to head (428435e).
Report is 1 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #1879 +/- ##
==========================================
- Coverage 68.98% 68.96% -0.02% 
==========================================
Files 66 66 Lines 5243 5243 ==========================================
- Hits 3617 3616 -1 - Misses 1395 1396 +1 
Partials 231 231 
FlagCoverage Δ
e2e50.46% <ø> (-0.08%)⬇️
unit56.70% <ø> (-0.02%)⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@anik120anik120 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

@openshift-ciopenshift-ciBot added the lgtm Indicates that a PR is ready to be merged. label Mar 26, 2025
@anik120
anik120 added this pull request to the merge queue Mar 26, 2025
Merged via the queue into operator-framework:main with commit a2ae8b8Mar 26, 2025
@grokspawn
grokspawn deleted the catalog-fetching-metas branch March 26, 2025 19:14
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtmIndicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[docs] update catalogd component-internal web api docs, demos

2 participants

@grokspawn@anik120