Skip to content

Fix version check#226

Open
haneug wants to merge 6 commits intofaccts:mainfrom
haneug:fix.version_check
Open

Fix version check#226
haneug wants to merge 6 commits intofaccts:mainfrom
haneug:fix.version_check

Conversation

@haneug
Copy link
Copy Markdown
Contributor

@haneug haneug commented Apr 2, 2026

Small fix to version check.

@haneug haneug self-assigned this Apr 2, 2026
@haneug haneug added the bug Something isn't working label Apr 2, 2026
@haneug haneug requested a review from a team as a code owner April 2, 2026 13:20
@haneug haneug force-pushed the fix.version_check branch from f9f3d87 to 066c0d7 Compare April 7, 2026 11:18
Copy link
Copy Markdown
Contributor

@timmyte timmyte left a comment

Choose a reason for hiding this comment

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

Great idea to fix this. I would how ever apply the fix slightly further down in the code on deeper level.

Comment thread src/opi/utils/orca_version.py Outdated
Comment thread src/opi/utils/orca_version.py Outdated
haneug and others added 3 commits April 8, 2026 08:57
@haneug
Copy link
Copy Markdown
Contributor Author

haneug commented Apr 15, 2026

Thanks I added your suggestions and also changed RGX_VERSION to allow the x in micro/path

@haneug haneug requested a review from timmyte April 15, 2026 07:48
Copy link
Copy Markdown
Contributor

@timmyte timmyte left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks for fixing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants