Skip to content

[core] Ignore code-infra-renovate[bot] commits in changelog - #23153

Merged
Janpot merged 1 commit into
mui:masterfrom
Janpot:fix-changelog-bot-filter
Jul 15, 2026
Merged

Janpot merged 1 commit into
mui:masterfrom
Janpot:fix-changelog-bot-filter

Conversation

@Janpot

@Janpot Janpot commented Jul 15, 2026

Copy link
Copy Markdown
Member

The changelog script guarded against the renovate bots inconsistently:

As a result, code-infra-renovate[bot] commits (e.g. dependency bumps like #23121) still showed up as changelog entries while being excluded from contributor attribution.

This extracts a shared IGNORED_BOT_LOGINS set used by both guards so they can't drift again, and treats code-infra-renovate[bot] the same as renovate[bot] — excluded from the changelog entirely.

@code-infra-dashboard

Copy link
Copy Markdown

Deploy preview

https://deploy-preview-23153--material-ui-x.netlify.app/
QR code for https://deploy-preview-23153--material-ui-x.netlify.app/

Bundle size

Bundle Parsed size Gzip size
@mui/x-data-grid 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-pro 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-premium 0B(0.00%) 0B(0.00%)
@mui/x-charts 0B(0.00%) 0B(0.00%)
@mui/x-charts-pro 0B(0.00%) 0B(0.00%)
@mui/x-charts-premium 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) 0B(0.00%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 0B(0.00%)
@mui/x-scheduler 0B(0.00%) 0B(0.00%)
@mui/x-scheduler-premium 0B(0.00%) 0B(0.00%)
@mui/x-chat 0B(0.00%) 0B(0.00%)
@mui/x-license 0B(0.00%) 0B(0.00%)

Details of bundle changes


Check out the code infra dashboard for more information about this PR.

@Janpot Janpot added type: regression A bug, but worse, it used to behave as expected. scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). labels Jul 15, 2026
@Janpot
Janpot marked this pull request as ready for review July 15, 2026 13:31
@Janpot
Janpot requested a review from a team July 15, 2026 13:32
@Janpot
Janpot merged commit 0c1559f into mui:master Jul 15, 2026
24 of 25 checks passed
@LukasTy

LukasTy commented Jul 15, 2026

Copy link
Copy Markdown
Member

This change would have been irrelevant after #23112, where deps PR filtering is based on labels ("dependencies" excluded).

LukasTy added a commit to LukasTy/mui-x that referenced this pull request Jul 15, 2026
…5ffb

Reconcile with mui#23153: keep its IGNORED_BOT_LOGINS set (used by getContributors for attribution), but drop the author filter it added on the commit stream. This branch already excludes dependency bumps via the fixed `dependencies` label check, which covers every Renovate PR and survives future bot renames, so the author filter is redundant.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). type: regression A bug, but worse, it used to behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants