Closed nitroxide-poisoning closed 3 years ago
Hey @NitronBeenGrinding, thanks for the contributions!
Generally for contributions, especially of new features, we recommend opening an issue first for discussion of the feature and the implementation strategy that works best for integration and maintenance.
There are a few separate ideas here that should get their own separate issues (if one does not exist) and PRs for consideration:
deon
package on POST
, and the features that make it useful would would be to expose the YML, make it editable, and then generate the output formats interactively based on the user edited YML. What is in the PR is basically the "rendered verisons" that we already link to from deon.drivendata.org: https://github.com/drivendataorg/deon/tree/master/examplesOther smaller issues with the PR:
*.pyc
files that should not be in source controlesp
as submitted)output
folder and generated ethics.md
from the PR (already in examples
)Hopefully that's enough to split this up into issues and PRs that we can tackle independently. I'll close this one for now.
Alright, sounds good. I will make the changes in the coming weeks.
Changed the following (at a high level):
If any questions arise, feel free to contact me at any time.