Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

General: Do not validate version if build does not support it #2557

Merged
merged 6 commits into from
Jan 19, 2022

Conversation

iLLiCiTiT
Copy link
Member

Brief description

Current check of updates does not expect that version check is not available for current openpype build.

Changes

  • skip version validation if build does not store OpenPypeVersion to sys.modules.

Testing notes:

  1. Run OpenPype zip from this branch in older OpenPype build

@iLLiCiTiT iLLiCiTiT self-assigned this Jan 19, 2022
@iLLiCiTiT iLLiCiTiT requested review from a user, kalisp and mkolar January 19, 2022 09:16
@iLLiCiTiT iLLiCiTiT added backend type: bug Something isn't working labels Jan 19, 2022
@iLLiCiTiT iLLiCiTiT requested review from antirotor and removed request for a user January 19, 2022 17:20
Copy link
Member

@antirotor antirotor left a comment

Choose a reason for hiding this comment

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

It does work... or I can't find side-effect. But I am not sure if I am testing it properly:

  • Created zip from this
  • Checked out some older verison like 3.7.0-nightly.12
  • Build it and run it like openpype_console --use-version=3.8.0-nightly.4
  • Encountered no errors

.github/workflows/test_build.yml Outdated Show resolved Hide resolved
Co-authored-by: Ondřej Samohel <33513211+antirotor@users.noreply.github.com>
@iLLiCiTiT iLLiCiTiT merged commit 0d319af into develop Jan 19, 2022
@mkolar mkolar deleted the bugfix/skip_version_check_on_older_builds branch January 24, 2022 20:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants