Skip to content

Trigger tests using PR comments#3850

Open
shralex wants to merge 1 commit into
mainfrom
shralex_ci_1
Open

Trigger tests using PR comments#3850
shralex wants to merge 1 commit into
mainfrom
shralex_ci_1

Conversation

@shralex
Copy link
Copy Markdown
Collaborator

@shralex shralex commented May 8, 2026

This PR adds a few comment commands: test this please / test this pr / run tests, to re-run PR tests. As well as: run scheduled tests to trigger the full "scheduled" test suite. The goal is to make it easier to rerun PRs, without going to the actions tab, which some users don't have permissions to view.

Checklist

Before submitting this PR, please make sure (put X in square brackets):

  • I have performed a self-review of my code. For an optional AI review, add the gemini-review label.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have run end-to-end tests tests and provided workload links above if applicable.
  • I have made or will make corresponding changes to the doc if needed, including adding new documentation pages to the relevant Table of Contents (toctree directive) as explained in our documentation.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@shralex
Copy link
Copy Markdown
Collaborator Author

shralex commented May 9, 2026

test this pr

- Add pr_comment_commands.yml to trigger standard or scheduled PR tests via issue comments.

- Simplify build_and_test_maxtext.yml test orchestration logic and inputs.

- Support CI_STEPS environment variable in post-training notebooks to dynamically scale down step count (e.g. to 2 steps) for fast CI smoke tests.

- Skip pathways unit tests in CI due to GHA service container TPU access limitations.
@shralex shralex force-pushed the shralex_ci_1 branch 3 times, most recently from afcf8fc to 7b317e3 Compare May 10, 2026 16:03
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