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

Adds auto yaml generation #2587

Merged
merged 19 commits into from
Sep 23, 2022
Merged

Adds auto yaml generation #2587

merged 19 commits into from
Sep 23, 2022

Conversation

colin99d
Copy link
Contributor

@colin99d colin99d commented Sep 22, 2022

Description

This created the additions to main.yml and also adds the folder leven _index.md files. These files are currently bare bones and will need to be manually improved. However, that is outside the scope of this PR.

How has this been tested?

Generated documentation and confirmed it works in Hugo

Checklist:

Others

  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • My code passes all the checks pylint, flake8, black, ... To speed up development you should run pre-commit install.
  • New and existing unit tests pass locally with my changes. You can test this locally using pytest tests/....

@colin99d colin99d marked this pull request as draft September 22, 2022 07:21
@colin99d colin99d added the enhancement Enhancement label Sep 23, 2022
@colin99d colin99d marked this pull request as ready for review September 23, 2022 10:03
@minhhoang1023
Copy link
Contributor

@JerBouma Can you have a look at this well and see if there's anything else to add?

@JerBouma
Copy link
Contributor

Ok, so if it auto-generates and creates pages like you see for the OpenBB Terminal I am happy to just approve this. All there was missing were the references in the main.yml.

@colin99d colin99d merged commit 0b33cb4 into main Sep 23, 2022
@colin99d colin99d deleted the fix_auto branch September 23, 2022 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants