## Reviewing PRs

Reviewers ensure MinIO’s commit history remains a clear, reliable record. Responsibilities include:

1. **Commit Quality**:
   - Verify each commit explains *why* the change was made (e.g., “So that…”).
   - Request rebasing if commits are unclear, redundant, or lack context (e.g., “Please squash typo fixes into the parent commit”).

2. **Code Quality**:

### Creating commits and writing commit messages

The commit messages that accompany your code changes are an important piece of documentation. Please follow these guidelines when creating commits:

* [Write good commit messages.](https://cbea.ms/git-commit/#seven-rules)

            System.out.println("Commit " + commit + " wrapper: " + wrapperVersion);
            if (!ALLOWED_WRAPPER_VERSION.matcher(wrapperVersion).matches()) {
                System.err.println(
                    "Bad wrapper version " + wrapperVersion + " used in commit " + commit
                        + ". Please rebase your branch to ensure that each commit uses only released Gradle versions in wrapper (GA, RC or milestone)."

     * commit was made.
     * <p>
     * This approach doesn't work with merge commits as these can introduce commits in the chronological order
     * after the fact e.x. a merge done today can add commits dated with yesterday so the result will no longer be
     * deterministic.
     * <p>

        if (args.length != 0) {
            System.err.println("Usage: java CheckBadMerge.java < commits.txt");
            System.exit(2);
        }

        List<String> commits = readCommitsFromStdin();

        try {
            for (String commit : commits) {
                checkCommit(commit);
            }
        } finally {
            THREAD_POOL.shutdown();
        }
    }

# This file contains SHAs of changes that should be ignored when e.g.
# running `git blame` on a file. Do not edit any of the existing commits.
#
# Use this file by running:
#
#     git blame --ignore-revs-file=.git-blame-ignore-rev <file>
#
# or by configuring `blame.ignoreRevsFile`. The latter ought to also work
# with IDEs such as IntelliJ.


# Format snapshot / restore directories in server
1afe4b914301a23fa37c41c78185b7575a431cc4

    )
    if not use_pr:
        logging.error(f"No PR found for hash: {settings.commit_sha}")
        return
    commits = list(use_pr.get_commits())
    current_commit = [c for c in commits if c.sha == settings.commit_sha][0]
    run_url = f"https://github.com/{settings.github_repository}/actions/runs/{settings.run_id}"
    if settings.state == "pending":

https://github.com/google/guava/commit/6a5ca217af46eff28bd9703aeba62f57c5a3693a
https://github.com/google/guava/commit/91fee0605ca75cf4f7d0b07ea3068f8beaedda46
https://github.com/google/guava/commit/8dca77634125cbaf53365000d1c9f7fb32f6dcea
https://github.com/google/guava/commit/dd5f97bf678b13564ffa1b4a45e00284fb24f178
https://github.com/google/guava/commit/d812e15c120fea9c7cf9cd33be21c9cd0e424f81

        uses: peter-evans/create-pull-request@v8
        with:
          commit-message: Update performance test durations
          branch: bot-update-performance-test-durations
          branch-suffix: timestamp
          delete-branch: true
          title: 'Update performance test durations'
          signoff: true
          sign-commits: true
          labels: |
            in:building-gradle
            @dev-productivity

     *
     * @return the execution time in milliseconds
     */
    long getExecuteTime();

    /**
     * Commits all pending index update operations to ensure data persistence.
     * This method should be called after all documents have been stored.
     */
    void commit();