Skip to content

📖 Simplify jq examples using olm.csv.metadata - #1464

Merged
m1kola merged 2 commits into
operator-framework:mainfrom
bentito:docs-fix-jq-allnamespaces
Nov 14, 2024
Merged

📖 Simplify jq examples using olm.csv.metadata#1464
m1kola merged 2 commits into
operator-framework:mainfrom
bentito:docs-fix-jq-allnamespaces

Conversation

@bentito

@bentitobentito commented Nov 14, 2024

Copy link
Copy Markdown
Contributor

Description

Think this works as desired. Check it please.

Fixes#1460

Reviewer Checklist

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

@bentito
bentito requested a review from a team as a code ownerNovember 14, 2024 13:54
@netlify

netlifyBot commented Nov 14, 2024

Copy link
Copy Markdown

Deploy Preview for olmv1 ready!

NameLink
🔨 Latest commit60a9501
🔍 Latest deploy loghttps://app.netlify.com/sites/olmv1/deploys/673608d2e697e40008b29f21
😎 Deploy Previewhttps://deploy-preview-1464--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.

@m1kolam1kola changed the title 🐛 Simplify jq examples using olm.csv.metadata📖 Simplify jq examples using olm.csv.metadataNov 14, 2024
Signed-off-by: Brett Tofel <btofel@redhat.com>
@bentito
bentitoforce-pushed the docs-fix-jq-allnamespaces branch from 05e057a to 3d88615CompareNovember 14, 2024 14:14

@m1kolam1kola 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.

This seems to work as expected.

We only need to fix formatting in docs/howto/catalog-queries.md which got lost in rebase, I think.

Comment threaddocs/howto/catalog-queries.md Outdated
Co-authored-by: Mikalai Radchuk <509198+m1kola@users.noreply.github.com>
@m1kola
m1kola added this pull request to the merge queue Nov 14, 2024
@m1kola

Copy link
Copy Markdown
Member

Looks good. Thank you!

Merged via the queue into operator-framework:main with commit 9c5c3caNov 14, 2024
Sign up for freeto 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.

Fix "Return list of packages that support AllNamespaces install mode and do not use webhooks" catalog query in the docs

2 participants

@bentito@m1kola