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

General: Remove repos related logic #3087

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Apr 22, 2022

Brief description

Remove logic related to repos directory in OpenPype which is removed in PR.

Description

Repos directory related logic was removed and few places based on repos directory path was modified. Repos directory is not handled at all from this moment. Zip files copied from version repository are removed.

Additional info

PR is extending this PR. This change may affect how OpenPype works so it was separated to this PR to see changes better.

Testing notes:

  • Test if OpenPype from code works
  • Zip can be created from code
  • Build can be created from code
  • Build can load zip from openpype repository
    • zip copied to Local App data should be removed when extracted

@ynbot
Copy link
Contributor

ynbot commented Apr 22, 2022

Task linked: OP-2858 move AvalonMongoDB logic

@iLLiCiTiT iLLiCiTiT self-assigned this Apr 22, 2022
@iLLiCiTiT iLLiCiTiT added the type: enhancement Enhancements to existing functionality label Apr 22, 2022
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.

I think we should clean it a little bit more

igniter/bootstrap_repos.py Outdated Show resolved Hide resolved
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.

works!

@iLLiCiTiT iLLiCiTiT merged commit 37f348f into enhancement/OP-2858_move-AvalonMongoDB-logic Apr 25, 2022
@iLLiCiTiT iLLiCiTiT deleted the enhancement/remove_repos_related_logic branch April 25, 2022 15:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants