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

[WIP] Entry points #78

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

[WIP] Entry points #78

wants to merge 24 commits into from

Conversation

dubhsuil
Copy link

Adds a setup.py file to the extension IOT properly expose entry points to the environment. Includes many older commits as I am piggy-backing off of the work of others. I am raising this as a WIP PR mainly to show the changes being made and invite any thoughts from reviewers.

This change may show the need for a greater code cleanup, as using setup.py circumvents some things added to files like __init__.py, but there is a discussion to be had on that.

nimbinatus and others added 11 commits April 10, 2018 11:18
* Continues from deconst#76
* Closes deconst#73
* Alpine 3.8
* Pip is now provided in base python3 package
* Placeholder to remove build deps for security and size
  (needs runtime testing)
* Use new Python3 venv builtin (pyvenv is deprecated)
* MAINTAINER is deprecated
* Use latest pip packages in requirements
* Update gitignore
… the entrypoints are properly exposed. There is plenty more cleanup to be done, but this gets things working.
… this is a result of my changes, or existing problems
@robb-romans robb-romans mentioned this pull request Nov 5, 2018
Patrick Kirchhoff and others added 13 commits November 5, 2018 15:22
…ires the name of our extension in order for sphinx to load it. Many small changes aside from that as I try to massage things into place.
…roducing inaccurate output. Work continues...
…ual copying of files is not yet working though.
…de related issues in the output namely the right quotation mark (\u2019) and the paragraph symbol (\u00b6). Also, in order to get assets working, I had to rewrite the functionality into "calculate_offsets" because the rest of writer was not able to properly initialize.
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

2 participants