Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PEP8 #7

Closed
wants to merge 8 commits into from
Closed

PEP8 #7

wants to merge 8 commits into from

Conversation

tobiasherp
Copy link

Called pep8ify -wn, and edited the result.
Most changes are the removal of trailing whitespace.
Alignment of comments recovered.

Backup files of editors (...) are ignored.

Tobias and others added 4 commits July 24, 2015 22:44
If a section tells the transmogrifier about the created archive
(by calling transmogrifier.add_info('export_content', name, info), e.g.
at the end of the __iter__ method), the calling method can return this
archive to the user for download. This way, transmogrifier turns into a
tool which can be offered to "normal" users for flexible exports,
without the need to edit the pipeline configuration first.
Here is all the meat!
Produces a pretty string which represents the _all attribute of a
Transmogrifier object.  Can return a list instead.

Conflicts:
	src/collective/transmogrifier/utils.py
When ConfigurationRegistry.clear is fired unexpectedly, this can go
unnoticed and cause transmogrifications to fail.  Thus, this can be
customized by setting the CLEANUP variable.

To suppress the call by the zope.testing.cleanup facility entirely, this
variable must must be changed to OFF in the module code, because this is
done on import time.  For other uses, the variable can be set after
importing.

The present SILENT value reflects the so-far behaviour.

Conflicts:
	src/collective/transmogrifier/transmogrifier.py
@thet thet mentioned this pull request Apr 14, 2016
@thet
Copy link
Member

thet commented Apr 14, 2016

Closed via: #10

@thet thet closed this Apr 14, 2016
@thet thet reopened this Apr 14, 2016
@thet
Copy link
Member

thet commented Apr 14, 2016

Have to reopen - I withdraw the attempt to rebase on your PR, as it is not only simple PEP8 but also adds new functionality.

@jensens
Copy link
Member

jensens commented Jul 9, 2021

perhaps outdated, reopen if i am wrong

@jensens jensens closed this Jul 9, 2021
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.

None yet

3 participants