Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Generated via commit 4dc8c82 Download link for the artifact containing the test results: ↓ atime-results.zip
|
| GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }} | ||
| repo_token: ${{ secrets.GITHUB_TOKEN }} | ||
| steps: | ||
| - uses: Anirban166/Autocomment-atime-results@v1.4.3 No newline at end of file |
There was a problem hiding this comment.
IIUC this is designed to supplant the old action which can't be updated? Is there anything stopping us from forking it instead (or, if the current PR is intended as such a fork, could we at least host it in a dedicated repo under Rdatatable or maybe under your tdhock?)
There was a problem hiding this comment.
right, the current PR is a fork. @Anirban166 would you consider putting this new code in your Autocomment-atime-results repo? (I was not sure if Ani is willing / able to continue maintenance, but that would be great)
The basic problem with the old action is that is uses an old cml container, old compilers, and this PR fixes that.
This reverts commit e1fa82b.

The old atime CI uses an old container (old compilers etc.)
I propose switching to this new CI which is definitely more code.
Plus: easier to customize.
Minus: more maintenance burden.