pyblish / pyblish-base

Pyblish base library - see https://github.com/pyblish/pyblish for details.
Other
127 stars 59 forks source link

Integrators #82

Closed mottosso closed 9 years ago

mottosso commented 10 years ago

For the 1.1, as is being discussed in the Usergroup, we're thinking about renaming Conformer to Integrator.

This issue is part of the 1.1 "milestone". Let's tag each issue relevant to the 1.1 with this milestone, that way, we can spot progress and keep track of exactly what we are up against.

mottosso commented 9 years ago

This has been implemented in the form of an alias, since 1.0.1.

import pyblish.api as pyblish

class IntegrateAsset(pyblish.Integrator):
  ...

However bear in mind that the physical files will still need to be named conform_.

Considering whether both should be allowed?

# E.g.
conform_asset.py
integrate_asset.py
tokejepsen commented 9 years ago

I'm actually warming to conforming:)

mottosso commented 9 years ago

:)

mkolar commented 9 years ago

I quite like conform too. But quite frankly renaming it doesn't bother me in the slightest. Both work equally good as far as I can tell.

mottosso commented 9 years ago

Just to clarify, there won't be any renaming. This would simply add another option to how you are able to name the files on disk. Either prefixed with conform_ or integrate_ depending on your preference.

You could potentially use both, depending on whether one suits better than the other.

conform_images.py
integrate_asset.py
mottosso commented 9 years ago

Implemented in 1.1