Skip to content

ci: run E2E tests against the open udata PR linked in the description - #1280

Merged
ThibaudDauce merged 2 commits into
mainfrom
ci/resolve-udata-ref
Oct 19, 2026
Merged

ThibaudDauce merged 2 commits into
mainfrom
ci/resolve-udata-ref

Conversation

@ThibaudDauce

@ThibaudDauce ThibaudDauce commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

  • Avoids the "switch back to main udata in CI" PRs (fix: udata branch in CI #1131, fix: switch back to main udata in CI #1271, fix: switch back to main udata in CI #1278): the udata ref no longer lives in ci.yml, so it can't land on main.
  • The E2E job checks out the first udata PR linked in the PR description (opendatateam/udata/pull/N) that is still open, main otherwise. Following links point to merged PRs, so they fall back to main on their own.
  • To ignore a udata PR you only mention, write it as opendatateam/udata#N.
  • After editing the description, re-run the jobs (or push an empty commit): the description is fetched at run time.

@nicolaskempf57

Copy link
Copy Markdown
Contributor

Can you try with an open udata PR in the description ? I'm not finding quality information about the scope of the token outside of the current organization

@nicolaskempf57 nicolaskempf57 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice ! The token seems to be reaching udata refs as expected

@ThibaudDauce
ThibaudDauce merged commit 575b66c into main Oct 19, 2026
28 of 29 checks passed
@ThibaudDauce
ThibaudDauce deleted the ci/resolve-udata-ref branch October 19, 2026 06:37
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.

2 participants