Update CI artifact naming

openshmem-org / specification

OpenSHMEM Application Programming Interface

http://www.openshmem.org

50 stars 32 forks source link

Update CI artifact naming #380

Closed nspark closed 4 years ago

nspark commented 4 years ago

I am also hoping to use this PR to test the new openshmem-bot.

openshmem-bot[bot] commented 4 years ago

Draft PDF in openshmem-draft-8d8bdfc403fa43219095909c5700936d0b2679dc.zip

nspark commented 4 years ago

Per https://developer.github.com/v3/checks/suites/, the check_suite response only contains linked pull_requests for PRs in the same project, not across forks. So, as currently implemented, the openshmem-bot can only provide the link to the draft spec in these cases.

jdinan commented 4 years ago

@nspark I don't see any reason why we should hold off on merging this. Is it ready to go?

nspark commented 4 years ago

Yep!