search

Anirban166 / Autocomment-atime-results

GitHub Action that automatically comments a plot and other atime-based results on PRs
https://github.com/marketplace/actions/autocomment-atime-results
0 stars 1 forks source link

Test artifact generation and download #18

Closed Anirban166 closed 8 months ago

Anirban166 commented 8 months ago

A test run for #17

github-actions[bot] commented 8 months ago

Comparison Plot

Generated via commit 8a8138f4c6e67b0072fc6af6950ce758f004ef6d

Download link for the artifact containing the test results: ↓ atime-results.zip

Anirban166 commented 8 months ago

@tdhock Like the example above, the results obtained after running atime::atime_pkg on the target repository will be ready to download with a click/tap now :)

tdhock commented 8 months ago

ok that kind of PR comment is great in principle but why is it showing up here? it should show up in a PR of an R package repo with inst/atime/tests.R file.

Anirban166 commented 8 months ago

it should show up in a PR of an R package repo with inst/atime/tests.R file.

It will once I'm done with the changes for my action to have it work inside another repository (without the need to clone it) in a way that atime can get the required environment variables for the branches. That should be the final step and once I get that working, we should get the plot versions with base and merge-base as well (apart from head and cran which currently shows up), and then you won't see a PR in this repository even!

I tried the tactic that Siyao used, namely her git-switch step but it is still throwing me the same error (please see #8 for reference).

The pull requests on this repository are to test specific parts of the bigger picture - like the task at hand is a combination of a few features requiring to work correctly together, so I'm isolating them, working on them individually, and making PRs for them case-by-case (with appropriate PR names I hope!) to just test one thing at a time.

For example:

(I hope this makes it clear)