Skip to content

Simpler ci test#126

Closed
tdhock wants to merge 47 commits into
mainfrom
simpler-ci-test
Closed

Simpler ci test#126
tdhock wants to merge 47 commits into
mainfrom
simpler-ci-test

Conversation

@tdhock

@tdhock tdhock commented Jun 12, 2026

Copy link
Copy Markdown
Owner

foo
this is the description of the PR, not the first comment.

@tdhock

tdhock commented Jun 12, 2026

Copy link
Copy Markdown
Owner Author

tests-preview-facet.png is the first comment from json

@tdhock

tdhock commented Jun 12, 2026

Copy link
Copy Markdown
Owner Author

example has

        env:
          REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}

and https://github.com/Rdatatable/data.table/blob/master/.github/workflows/performance-tests.yml works with:

    env:
      GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}
      repo_token: ${{ secrets.GITHUB_TOKEN }}

@tdhock

tdhock commented Jun 12, 2026

Copy link
Copy Markdown
Owner Author
curl -L   -H "Accept: application/vnd.github+json"   -H "X-GitHub-Api-Version: 2026-03-10"   https://api.github.com/repos/tdhock/atime/issues/126/comments

list comments in this PR https://docs.github.com/en/rest/issues/comments?apiVersion=2026-03-10#list-issue-comments

@tdhock

tdhock commented Jun 12, 2026

Copy link
Copy Markdown
Owner Author

https://cml.dev/doc/ref/comment#update says Update the last CML comment instead of creating a new one. If no previous comment is found, create a new one.

--watermark-title=<...>: Hidden comment marker

pr comment Rdatatable/data.table#7789 (comment) has the following at the end

![](https://cml.dev/watermark.png#554ef507e2bf8ecb496e234071820c998af88f45 "CML watermark")

@tdhock

tdhock commented Jun 12, 2026

Copy link
Copy Markdown
Owner Author
curl -L   -X PATCH   -H "Accept: application/vnd.github+json"   -H "Authorization: Bearer <YOUR-TOKEN>"  -H "X-GitHub-Api-Version: 2026-03-10"  https://api.github.com/repos/OWNER/REPO/issues/comments/4694905150
  -d '{"body":"Me too"}'

Toby Dylan Hocking added 2 commits June 12, 2026 16:11
@tdhock tdhock marked this pull request as draft June 12, 2026 20:12
@tdhock tdhock closed this Jun 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant