Conversation
|
Getting caught up, playing with the integrated tools. Thanks @edhartnett for your patience. |
|
Howdy @WardF! Does the Copilot review get posted publicly like a old-fashioned human review? Or does it take a long time? I don't see anything... Presumably it would work best by posting its comments to the PR without you having to be involved, so that the PR author could address review issues. |
|
@edhartnett It should be public, but there are some questions to be answered internally vis-a-vis flipping the script to enable them. |
|
OK, it failed anyway. ;-) |
|
@WardF if you could merge this one next, it would be helpful. No code changes just docs. |
Modify section names to not collide.
|
Updated a file to avoid a section name collision that was introduced, looks good, will merge once tests finish running. |
|
Requested another copilot review to see if the failure repeats. Looks like it's working. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Fixes #3266
These changes were generated with AI assistance but with human in the loop (me!) for every decision. I have personally reviewed every line of change in the PR and it is all of benefit to netcdf-c. These changes represent a serious attempt to improve netCDF and are in accordance with project practices and customs.