feat: support runReport.json summaries, depend on @moonrepo/* packages - #10
Conversation
|
@suin @kahirokunn any chances this could have a merge? |
|
As this PR feats runtime dependencies, and most CI runners do not install dependencies prior to executing scripts, it is not ready for merge. I'll push a featuring of rollup in just a second. |
|
Should be good to go, see https://github.com/0xar-ds/public/actions/runs/17442145519/job/49527549154?pr=28#step:6:1 |
|
Thank you for your PR. Could you check the CI result? |
Well, that's because the snapshots are of my local machine & now they include tracing information of whether the action will summarize a ciReport or a runReport. We can go two ways from here, up for you to choose: a. Tests/CI workflow builds a snapshot of the program prior to testing against it I'd argue to go for A, which is my following commit. Let me know if you'd prefer the other option. |
|
Thank you for checking the CI result. I think option B is better than A, since that is testable and relative paths is conciser for users. Could you fix it? |
fix: @moonrepo/types is not a runtime dependency
…so infer summaries from runReport.json
chore: update tsconfig.json chore: update release destination chore: bump yarn lock
style: format file chore: bump yarn lock
|
@suin should be good to merge, tests are against the static snapshots present on the repository & it's passing on my branch |
|
Thank you! I've merged your great work. |
2 participants
closes #8, #1
style and functionality remains 1:1 with current release, though https://github.com/appthrust/moon-ci-retrospect/compare/main...0xar-ds:moon-ci-retrospect:main?expand=1#diff-dcdc3e0b3362edb8fec2a51d3fa51f8fb8af8f70247e06d9887fa934834c9122L121-R143 outputs new badges according to match the latest type of ActionStatus