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

Separate building documentations and notebook execution testing #20

Merged
merged 3 commits into from
Dec 14, 2023

Conversation

lbianchi-lbl
Copy link
Contributor

Addresses Issue: #18

Changes proposed in this PR:

  • Configure Jupyter Book to render notebooks as-is (i.e. without executing them)
  • Add separate testing and CI harness to test notebooks that can be run non-interactively (e.g. excluding tutorial exercises)

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the license terms described in the LICENSE.md file
    at the top level of this directory.

  2. I represent I am authorized to make the contributions and grant the license. If my employer has
    rights to intellectual property that includes these contributions, I represent that I have
    received permission to make contributions and grant the required license on behalf of that
    employer.

@lbianchi-lbl
Copy link
Contributor Author

I'm merging this into main to test whether it's enough to make RTD happy.

@lbianchi-lbl lbianchi-lbl marked this pull request as ready for review December 14, 2023 21:10
@lbianchi-lbl lbianchi-lbl self-assigned this Dec 14, 2023
@lbianchi-lbl lbianchi-lbl merged commit 881bc29 into prommis:main Dec 14, 2023
24 checks passed
dangunter added a commit to dangunter/prommis that referenced this pull request Dec 15, 2023
* moved all code under a package dir
* install instructions
@ksbeattie ksbeattie added the Priority:Normal Normal Priority Issue or PR label Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:Normal Normal Priority Issue or PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants