Skip to content

Add conda packaging#64

Open
Zehen-249 wants to merge 1 commit into
Aditya-Donde:devfrom
Zehen-249:dev
Open

Add conda packaging#64
Zehen-249 wants to merge 1 commit into
Aditya-Donde:devfrom
Zehen-249:dev

Conversation

@Zehen-249

Copy link
Copy Markdown

Ospgrillage packeged as submodule

Comment thread src/osdagbridge/__init__.py Outdated
Comment thread src/osdagbridge/desktop/__init__.py Outdated
Comment thread src/osdagbridge/desktop/__main__.py

@mhsuhail00 mhsuhail00 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Recheck the comments

@Zehen-249

Copy link
Copy Markdown
Author

Checks Done

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Zehen-249
This refactoring changes may not be required in this PR.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

that caused issue during packaging, that's why refactored

@mhsuhail00

Copy link
Copy Markdown
Collaborator

The ospgrillage module should be a submodule i think. But it is not so.

@Zehen-249

Copy link
Copy Markdown
Author

The ospgrillage module should be a submodule i think. But it is not so.

osdagbridge - module,
ospgrillage - module

both packaged together in osdagbridge module, in future when we want to remove ospgrillage dependency we just have to remove the folder.

ospgrillage is not a submodule in sense of submodule theory.

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