search

InsightSoftwareConsortium / ITKRemoteModuleBuildTestPackageAction

A composite GitHub Action to build, test, and package, ITK remote modules
Apache License 2.0
3 stars 6 forks source link

DOC: Update README and add input descriptions #30

Closed tbirdso closed 1 year ago

tbirdso commented 1 year ago

Addresses https://github.com/InsightSoftwareConsortium/ITKRemoteModuleBuildTestPackageAction/issues/28

tbirdso commented 1 year ago

The updated README with markdown can be viewed here: https://github.com/InsightSoftwareConsortium/ITKRemoteModuleBuildTestPackageAction/blob/4f466d59cd207ee472d3563c3f19e9821935b667/README.md

dzenanz commented 1 year ago

I find it more useful to look at the rich diff: https://github.com/InsightSoftwareConsortium/ITKRemoteModuleBuildTestPackageAction/pull/30/commits/4f466d59cd207ee472d3563c3f19e9821935b667?short_path=b335630#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

dzenanz commented 1 year ago

Ah, I now see that almost the whole file is new 😄

tbirdso commented 1 year ago

I find it more useful to look at the rich diff: 4f466d5?short_path=b335630#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

Great tip. I will do this in the future.

tbirdso commented 1 year ago

@jhlegarreta Thanks for the feedback. I think I've addressed all of the points you noted, please let me know if anything is missing.

jhlegarreta commented 1 year ago

@jhlegarreta Thanks for the feedback. I think I've addressed all of the points you noted, please let me know if anything is missing.

Looks good. Thanks Tom. Squash and merge the WIP commits into the README DOC commit and it'd be ready to go for me.

tbirdso commented 1 year ago

Squashed changes

thewtex commented 1 year ago

@tbirdso thanks!