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

Maya: Change mayaAscii family to mayaScene #2106

Merged
merged 6 commits into from
Oct 6, 2021

Conversation

karimmozilla
Copy link
Contributor

No description provided.

@mkolar
Copy link
Member

mkolar commented Oct 5, 2021

@@ -3,14 +3,14 @@
import pyblish.api


class CollectMayaAscii(pyblish.api.InstancePlugin):
class CollectMayaScene(pyblish.api.InstancePlugin):
Copy link
Member

Choose a reason for hiding this comment

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

This class name is already used for collecting the workfile itself and because of the clash publishing doesn't work. It is, however more of a problem of the other plugin where the name doesn' match the actual plugin python file name, so I reckon let's rename the class in collect_scene.py to CollectWorkfile and rename the same file to match other hosts collect_workfile.py

When that's done we can leave this class as is COllectMayaScene but let's please match the filename too. from collect_mayaascii.py to collect_mayascene.py

Copy link
Member

Choose a reason for hiding this comment

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

Should it be collect_maya_scene.py?

Copy link
Member

Choose a reason for hiding this comment

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

actually yes, it should

@mkolar mkolar changed the title feature/PYPE-1131 Change maya ascii family to maya scene Maya: Change mayaAscii family to mayaScene Oct 6, 2021
@mkolar mkolar self-requested a review October 6, 2021 11:33
@mkolar mkolar merged commit fadab4e into develop Oct 6, 2021
@mkolar mkolar deleted the Change-mayaAscii-family-to-mayaScene branch October 6, 2021 15:28
@mkolar mkolar added type: feature Larger, user affecting changes and completely new things deprecated type: enhancement Enhancements to existing functionality and removed type: feature Larger, user affecting changes and completely new things labels Oct 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
deprecated type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants