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

🐛 Fix XDG_DATA_HOME build issue#264

Merged
everettraven merged 1 commit intooperator-framework:mainfrom
trgeiger:bin-dir-fix
May 23, 2024
Merged

🐛 Fix XDG_DATA_HOME build issue#264
everettraven merged 1 commit intooperator-framework:mainfrom
trgeiger:bin-dir-fix

Conversation

@trgeiger
Copy link
Copy Markdown
Contributor

@trgeiger trgeiger commented May 7, 2024

This mirrors the same fix made previously in operator-controller: operator-framework/operator-controller#790

That same fix was updated/fixed here: operator-framework/operator-controller#804
This PR reflects the latter, fixed version.

This mirrors the same fix made previously in operator-controller: operator-framework/operator-controller#790

Signed-off-by: Tayler Geiger <tayler@redhat.com>
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 7, 2024
@trgeiger trgeiger marked this pull request as ready for review May 7, 2024 20:22
@trgeiger trgeiger requested a review from a team as a code owner May 7, 2024 20:22
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 7, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented May 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 48.84%. Comparing base (250e348) to head (a10a89b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #264   +/-   ##
=======================================
  Coverage   48.84%   48.84%           
=======================================
  Files           8        8           
  Lines         434      434           
=======================================
  Hits          212      212           
  Misses        201      201           
  Partials       21       21           

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

Copy link
Copy Markdown
Collaborator

@everettraven everettraven left a comment

Choose a reason for hiding this comment

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

LGTM - thanks @trgeiger ! Apologies for the delayed review!

@everettraven everettraven added this pull request to the merge queue May 23, 2024
Merged via the queue into operator-framework:main with commit fae2442 May 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants