Skip to content

Conversation

@Ahzyuan
Copy link
Member

@Ahzyuan Ahzyuan commented Jun 24, 2025

Prep Steps

Before we dive in, let's confirm you've unlocked all these steps

  • πŸ” I've checked for duplicate PRs (click to check)
  • πŸ“œ I've followed contribution guidelines
    • ✨ I've linted and formatted my code (how?)
    • πŸ”§ I've formatted the PR title appropriately (how?)
    • 🎯 I've targeted the master branch (how?)
    • πŸ‘ I've allowed maintainer edits on my branch (how? β†’ See step 6)
  • πŸ›‘οΈ I've added/updated tests to cover my changes if prossible (how?)
  • πŸ“ I've added/updated the documentation for any new logic introduced (how?)
  • ⏳ I understand @Ahzyuan has been busy lately and may need time to review. (we appreciate your patience!)

Type of changes

  • 🐞 Bug fix
  • 🌟 New feature
  • πŸ§ͺ Tests
  • πŸ— Refactoring
  • ⚑ Performance improvement
  • πŸ“– Documentation / docstrings
  • ↩️ Revert
  • 🧩 Other

Description

Purpose

This PR is to change the type of test triggered by PR from compatibility test to the minimal test, so as to diagnose bugs as soon as possible. Meanwhile, using the minimal test helps to save my Github Actions quota.

Changes:

  • .github/workflows/compatibility_test.yml: delete the PR trigger conditions.
  • .github/workflows/minimal_test.yml: add the PR trigger conditions which are set in the compatibility_test.yml at past.
  • docs/src/en/contribute/conventions.md: add ci as a valid branch name prefix.
  • docs/src/en/contribute/prs.md, docs/src/en/others/management.md: update all contents related to both the types of test in the documentation.

Ahzyuan added 2 commits June 23, 2025 18:13
- **compability test**: only trigger manually before mergeing
- **minimal test**: triggered in most of the cases to quickly locate bugs
@Ahzyuan Ahzyuan self-assigned this Jun 24, 2025
@github-actions github-actions bot added the ci Changes related to continuous integration (CI) configuration or workflows. label Jun 24, 2025
@Ahzyuan Ahzyuan merged commit e83482a into master Jun 24, 2025
1 check passed
@Ahzyuan Ahzyuan deleted the ci/change-PR-trigger-test branch June 24, 2025 13:25
@Ahzyuan Ahzyuan changed the title ci: Change PR trigger test from compability test to minimal one ci: Change PR trigger test from compatibility test to minimal one Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes related to continuous integration (CI) configuration or workflows.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants