docs: clarify pull request CI progress - #5
Open
RidhaOracle wants to merge 4 commits into
Open
Conversation
Description: Updated CONTRIBUTING.md to include
code and non-code contributions.
Change-Id: I6a20ca80155434a46203f922d77da25d49186242
Problem: ======== The GitHub contribution path lacked consistent intake templates, shared local CI entry points, and automated build, test, formatting, ownership, and triage feedback. Pull requests could also define workflow changes that executed before those definitions were merged into trunk. The selected-actions policy prevented the external Codex action from running. Solution: ========= Add structured issue and pull request templates, CODEOWNERS-based review routing, path-based area labels, and an expanded contributor guide while preserving the existing non-code contribution terms. Add scripts/ci helpers for Ubuntu toolchain setup, Ninja and ccache builds, clang-format, and the default MTR test selection. Run automatic PR automation from trunk with pull_request_target. Verify PR merge parents, disable persisted checkout credentials, scope permissions per job, restore caches without saving PR-controlled data, and execute build and MTR helpers from the trusted base SHA. Build Debug configurations with GCC and Clang, run default MTR with six-hour limits, publish statuses on the PR head, assign code owners after OCA verification, reconcile approved PRs daily, and apply PR-only stale handling. Change-Id: I75d87674e369b28a7ac6707ebb796e0d2a8bcd83
1. Change summaryUpdates 2. Review findingsNo high-confidence, actionable findings. 3. Test gaps or risksDocumentation-only change with minimal risk. No functional tests are necessary; Markdown rendering should be verified in the repository host’s preview. |
1. Change summary
2. Review findings
3. Test gaps or risks
|
github-actions
Bot
added
Build Passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this change do?
Clarifies that contributors can use a pull request's Checks tab to follow live CI progress.
Why is it needed?
Provides a small, documentation-only change to exercise the pull-request automation.
How was it tested?
mysql-test/scripts/ci/mtr.shpasses locallygit diff --checkContributor checklist
AI assistance
Codex made the documentation-only clarification; the patch was reviewed with
git diff --check.Areas touched
Documentation and CI contributor guidance.