openedx / aspects-dbt

The dbt project for Open edX Aspects!
Apache License 2.0
2 stars 5 forks source link

chore: add dbt-coverage report #51

Closed Ian2012 closed 7 months ago

Ian2012 commented 7 months ago

Description

This PR modifies the documentation workflow to add a check with documentation coverage. Eventually, it can be modified to verify test coverage.

This PR also adds documentation for every dataset.

openedx-webhooks commented 7 months ago

Thanks for the pull request, @Ian2012! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

bmtcril commented 7 months ago

I think maybe coverage.json should be added to .gitignore? I know for other coverage things it's an artifact file that gets generated as part of the run but isn't kept around between runs.

openedx-webhooks commented 7 months ago

@Ian2012 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.