diff --git a/.mergify.yml b/.mergify.yml index ece2fc223c..09f12f908d 100644 --- a/.mergify.yml +++ b/.mergify.yml @@ -44,7 +44,7 @@ merge_protections: - label != hotfix success_conditions: - or: &DefaultReviewCond - - "#approved-reviews-by >= 2" + - "#approved-reviews-by >= 1" - author = dependabot[bot] - author = renovate[bot] # mergify-ci-bot needs no approval for the deterministic syncers, but @@ -57,22 +57,7 @@ merge_protections: if: - label = hotfix success_conditions: - - or: - - or: &HotfixReviewCond - # Author is engineering manager, we allow with one review - - and: - - author=@eng-mgr - - "#approved-reviews-by>=1" - - and: - - author=@devs - # in any other case, we allow if approve by engineering manager - # or anyone if it's a regular fix/revert - - or: - - approved-reviews-by=@eng-mgr - - and: - - title~=^(revert|fix) - - "#approved-reviews-by>=1" - - or: *DefaultReviewCond + - or: *DefaultReviewCond # yamllint disable-line rule:line-length - body ~= (Fix|Related).* (INC-\d+|https:\/\/github\.com\/Mergifyio\/docs\/security\/dependabot\/\d+) @@ -116,8 +101,8 @@ pull_request_rules: conditions: - -author=dependabot[bot] - -author=renovate[bot] - # Bots are not routed for review, except the docs agent, which now needs - # two approvals like anyone else. Gated on -draft so the ten drafts a + # Bots are not routed for review, except the docs agent, which needs + # approval like anyone else. Gated on -draft so the ten drafts a # Monday audit opens do not each ping the team before a human wants them. - or: - -author=mergify-ci-bot @@ -129,13 +114,8 @@ pull_request_rules: - and: *CheckRuns - "#changes-requested-reviews-by=0" - review-requested!=@devs - - and: - - or: - - label!=hotfix - - not: - or: *HotfixReviewCond - - not: - or: *DefaultReviewCond + - not: + or: *DefaultReviewCond actions: request_reviews: teams: diff --git a/src/content/docs/billing.mdx b/src/content/docs/billing.mdx index aacf79f322..977101a62f 100644 --- a/src/content/docs/billing.mdx +++ b/src/content/docs/billing.mdx @@ -4,8 +4,6 @@ description: Understand how Mergify fair billing works. --- import DownloadLink from '../../components/DownloadLink.astro'; -import { Image } from "astro:assets" -import PlanScreenshot from "../images/billing/plan.png" This page explains how Mergify's contributor-based billing works and how to read your invoice. @@ -103,16 +101,26 @@ invoiced for 10 extra users over 45 days. You can check your current usage on the [Plan page](https://dashboard.mergify.com/settings/plan) of the Mergify dashboard, -under **Settings → Plan**. It lists each product alongside its repository count -and its number of active contributors. +under **Settings → Plan**. It lists each product you use in a table, with its +repository count and its number of active contributors: -Settings Plan page listing each Mergify product with its repository and active contributor counts +| Product | Repositories | Active contributors | +|---|---|---| +| Merge Queue | 10 repositories | 25 contributors | +| CI Insights | 5 repositories | 15 contributors | +| Workflow Automation | 10 repositories | 25 contributors | + +The counts above are examples. If you are a billing manager, the table has an +**On-demand Usage** column showing what you are billed monthly on top of your +plan. On a yearly subscription, it also has a **Committed** column showing the +seats you pre-purchased. + +On some plans the third column is headed **Collaborators** instead. It counts +everyone with access to a repository, rather than only the people who +contributed to it. -You can then click on the number of contributors to get the list of active -users for each product. +Both counts are links. The repository count opens the list of repositories using +that product, and the contributor count opens the list of people counted for it. :::note This is not available for on-premise plans. diff --git a/src/content/docs/merge-queue/batches.mdx b/src/content/docs/merge-queue/batches.mdx index f47aad1e74..8b25cfed5d 100644 --- a/src/content/docs/merge-queue/batches.mdx +++ b/src/content/docs/merge-queue/batches.mdx @@ -738,7 +738,10 @@ temporary batch PRs, only when all of the following are true: Optionally set `update_bot_account` to avoid in-place updates blocked by GitHub for security reasons (for example, PRs from forks that modify workflows, or PRs -opened by other bots). +opened by other bots). The reverse holds for ordinary PRs from forks: with +`update_method: rebase`, setting the account makes Mergify refuse them at queue +entry. See [Combining Merge and Update +Methods](/merge-queue/merge-strategies#combining-merge-and-update-methods). ## Skip intermediate results (anti-flake protection) diff --git a/src/content/docs/merge-queue/direct-merge.mdx b/src/content/docs/merge-queue/direct-merge.mdx index ed24094094..dc36048611 100644 --- a/src/content/docs/merge-queue/direct-merge.mdx +++ b/src/content/docs/merge-queue/direct-merge.mdx @@ -108,8 +108,6 @@ request and a full CI run. - **The queue does not use [two-step CI](/merge-queue/two-step).** Two-step CI defers the heavy suite to the queue on purpose, so the pull request's own CI is not the full signal. -- **`autosquash` is off.** Rewriting the commits changes what was tested. - - **The merge method is not [`merge-batch`](/merge-queue/merge-strategies#merge-batch).** It merges the batch pull request itself, so there has to be one. diff --git a/src/content/docs/merge-queue/merge-strategies.mdx b/src/content/docs/merge-queue/merge-strategies.mdx index 577719c8ed..98311b89ce 100644 --- a/src/content/docs/merge-queue/merge-strategies.mdx +++ b/src/content/docs/merge-queue/merge-strategies.mdx @@ -158,10 +158,12 @@ rebase update occurs, the commit SHAs on the PR will change; what fast-forward preserves are the SHAs of the PR branch at merge time. :::caution - When using `update_bot_account` with fast-forward inplace mode, support for - fork pull requests is deprecated. See the - [update method deprecation note](#combining-merge-and-update-methods) for - details and migration options. + Combining fast-forward inplace mode with an `update_bot_account` makes + Mergify refuse pull requests coming from a fork when they are queued. + `update_method: merge` is not a way out here: a fast-forward queue lands by + moving the base branch onto the pull request's head, and it exists to keep + merge commits out of that history. See [Combining Merge and Update + Methods](#combining-merge-and-update-methods) for what to do instead. ::: 0`) or does not have a linear -history (`-linear-history`). With `autosquash` enabled, holding more than one -commit (`#commits > 1`) is a third way to satisfy that either/or requirement. +history (`-linear-history`). Rebasing writes the pull request's head branch, so the [restrictions on writing a fork's branch](/workflow/actions/update#fork-pull-requests) apply here too. diff --git a/src/content/images/billing/invoice-next-period.png b/src/content/images/billing/invoice-next-period.png deleted file mode 100644 index 43cb227703..0000000000 Binary files a/src/content/images/billing/invoice-next-period.png and /dev/null differ diff --git a/src/content/images/billing/invoice-previous-period.png b/src/content/images/billing/invoice-previous-period.png deleted file mode 100644 index 74589ef7c9..0000000000 Binary files a/src/content/images/billing/invoice-previous-period.png and /dev/null differ diff --git a/src/content/images/merge-protections/freeze-main.png b/src/content/images/merge-protections/freeze-main.png deleted file mode 100644 index 0d92164d13..0000000000 Binary files a/src/content/images/merge-protections/freeze-main.png and /dev/null differ diff --git a/src/content/images/stacked-gh-pr.png b/src/content/images/stacked-gh-pr.png deleted file mode 100644 index 6030c78bbf..0000000000 Binary files a/src/content/images/stacked-gh-pr.png and /dev/null differ diff --git a/src/content/images/stacked-mergify-pr.png b/src/content/images/stacked-mergify-pr.png deleted file mode 100644 index f04ed18a27..0000000000 Binary files a/src/content/images/stacked-mergify-pr.png and /dev/null differ diff --git a/src/content/images/workflow/writing-your-first-rule/config-editor.png b/src/content/images/workflow/writing-your-first-rule/config-editor.png deleted file mode 100644 index 474c489bef..0000000000 Binary files a/src/content/images/workflow/writing-your-first-rule/config-editor.png and /dev/null differ diff --git a/src/content/images/workflow/writing-your-first-rule/summary.png b/src/content/images/workflow/writing-your-first-rule/summary.png deleted file mode 100644 index 31e576c2e8..0000000000 Binary files a/src/content/images/workflow/writing-your-first-rule/summary.png and /dev/null differ