Skip to content

fix(rla): immediate fix for PR #5153 error#5172

Merged
lgritz merged 3 commits intoAcademySoftwareFoundation:mainfrom
lgritz:lg-5154fix
Apr 30, 2026
Merged

fix(rla): immediate fix for PR #5153 error#5172
lgritz merged 3 commits intoAcademySoftwareFoundation:mainfrom
lgritz:lg-5154fix

Conversation

@lgritz
Copy link
Copy Markdown
Collaborator

@lgritz lgritz commented Apr 29, 2026

Caught in review, but I didn't see the comment before I merged.

lgritz added 2 commits April 29, 2026 11:54
Signed-off-by: Larry Gritz <lg@larrygritz.com>
This PR crossed streams with another that turned on more warnings for
unchecked errors. The check isn't necessary here, though.

Signed-off-by: Larry Gritz <lg@larrygritz.com>
@lgritz lgritz requested a review from jessey-git April 29, 2026 19:06
Signed-off-by: Larry Gritz <lg@larrygritz.com>
@lgritz
Copy link
Copy Markdown
Collaborator Author

lgritz commented Apr 29, 2026

To make it pass CI, this incorporates the fixes that are being reviewed separately in #5171. Let's be sure to merge 5171 BEFORE merging this one to keep them straight.

I'm about to get on a plane, not sure if I'll have internet. In the interest of not having a broken main, can somebody with merge permissions please keep an eye on the CI here (and 5171) and merge 5171, and THEN this one, assuming that both CI runs pass ok? Thanks.

@lgritz lgritz merged commit d1de954 into AcademySoftwareFoundation:main Apr 30, 2026
30 checks passed
@lgritz lgritz deleted the lg-5154fix branch April 30, 2026 05:02
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Apr 30, 2026
…cademySoftwareFoundation#5172)

Caught in review, but I didn't see the comment before I merged.

---------

Signed-off-by: Larry Gritz <lg@larrygritz.com>
Sign up for free to 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.

2 participants