access-ci-org / Integration_Roadmaps

ACCESS Infrastructure Integration Roadmaps
0 stars 9 forks source link

differentiate between OnDemand's required and optional steps. #45

Closed johrstrom closed 1 year ago

johrstrom commented 1 year ago

Fixes #42

mollycule commented 1 year ago

From a markdown/readthedocs perspective this lgtm!

@johrstrom, is there any additional reviewer on the OOD side that you'd want added to this PR? If not, then I will submit an official review approval and merge this PR with the "squash and merge" option.

johrstrom commented 1 year ago

@johrstrom, is there any additional reviewer on the OOD side that you'd want added to this PR? If not, then I will submit an official review approval and merge this PR with the "squash and merge" option.

No I don't think so, but that's not to say it's perfect! We meet with ACCESS folks in a working group and I'm sure to get feedback there.

Which is to say, we're ready from the OnDemand side, but there could be more updates in the future.

mollycule commented 1 year ago

Works for me! Hey, the beauty of shifting to the git workflow is to deploy smaller changes more frequently, so further updates can happen in new PRs :tada: