why release check said it is not needed?

nipy / heudiconv

Flexible DICOM conversion into structured directory layouts

https://heudiconv.readthedocs.io

Other

232 stars 125 forks source link

why release check said it is not needed? #742

Open yarikoptic opened 6 months ago

yarikoptic commented 6 months ago

@jwodder any immediate idea why

even though we had many PRs merged and recent one then had release label?

jwodder commented 6 months ago

@yarikoptic It's not saying that a release isn't needed; the auto version command is simply erroring for an unknown reason. I don't believe trying to get debugging information with -v would help, as, last time I checked, auto sent all its debug information to stdout instead of stderr, so it'd all get captured in the version variable.

yarikoptic commented 6 months ago

eh, locally for me it seems to be ok:

❯ GH_TOKEN=$(git config hub.oauthtoken) intuit-auto version
patch

but I guess we just should (always) run ~/auto version -v before without capturing into variable to see what is wrong if anything goes wrong. ... will submit PR for that with release label.