-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update CI/CD to fix PyPi deployment failure and improve documentation deployment #58
Comments
@nutjob4life note the added task above for |
@jordanpadams I would like that too. When time pressure isn't an issue, the way I like to work is:
This way, you get a monotonically increasing suite. Downloading on demand is nice but can present intermittent test failures, esp. in automated nightly test environments. If the bundles aren't too large, I sure don't mind including them in the repository. We can omit them from source distributions with the caveat to end-users that "test suite will run in degraded mode" or similar. What do you think @jordanpadams? |
@nutjob4life i will leave it to you. I am definitely OK with this approach, I just know it can be time consuming. so whatever you think is best. at minimum, I would like us to have at least one regression test that runs on a bundle, outputs the AIP and SIP, and do some simple comparisons to ensure the outputs meet some success critieria (e.g. number of lines in .TABs, LIDs, some other maybe useful info) |
Hey @jordanpadams I am completely out of steam since resolving #71 for the PDS.20 milestone. However the PR for it #72 contains much of the work for the last checkbox on this issue above, Improved Regression Suite. That PR includes:
There's a easy-to-plug-in framework in there now to add additional bundles with various parameters. For this issue, then, that just leaves tagging dev releases which @tloubrieu-jpl may already have implemented. I need to take a closer look at the various GitHub workflows he's added to this repository. And then deploying dev docs. Upshot is, I've missed this issue for this milestone. I'm sorry 🙇 |
@nutjob4life no problem. thanks for getting to #71 ! that is totally OK for this build. this issue is not time sensitive. the other one was so we can get moving on operational release. @tloubrieu-jpl is working on deploying dev vs operational docs so he can probably chime in here and may even wrap up this issue for you. |
Update CI/CD to deploy in-dev is difficult to implement because there is not action to push into a specific directory in gh-pages. It will requires more work and discussion on how to handle automated documentation for software. This will be done in ticket #74 |
Note to self @nutjob4life: "changing your point of view" might be a good way to make headway on this effort, similar to how the "corral" does it. For a common approach that can span repositories, we may want to investigate making our own GitHub Action that can be plugged into multiple workflows without mushrooming each to 100s of lines. |
Tasks
Update CI/CD to deploy in-dev documentation to some other online directory (maybe https://nasa-pds-incubator.github.io/pds-deep-archive/development or https://nasa-pds-incubator.github.io/pds-deep-archive/dev?)Factored out into Organize the automated documentation generation for dev releases #74 for a project-centric approachThe text was updated successfully, but these errors were encountered: