Skip to content
This repository was archived by the owner on Mar 3, 2025. It is now read-only.

Conversation

@rashmigottipati
Copy link
Member

Fix printcolumn for lastUnpacked based on json tag and not the field name

Fixes #306

…eld name

Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com>
@rashmigottipati rashmigottipati requested a review from a team as a code owner August 30, 2024 15:55
@codecov
Copy link

codecov bot commented Aug 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 34.27%. Comparing base (2fc8018) to head (4853851).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #361   +/-   ##
=======================================
  Coverage   34.27%   34.27%           
=======================================
  Files          15       15           
  Lines         674      674           
=======================================
  Hits          231      231           
  Misses        417      417           
  Partials       26       26           

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

Copy link
Contributor

@ankitathomas ankitathomas left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Aug 30, 2024
@everettraven everettraven added this pull request to the merge queue Aug 30, 2024
Merged via the queue into operator-framework:main with commit 2a5c859 Aug 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

capture catalog content timestamp in ClusterCatalogStatus

3 participants