mui/base-uiPublic

NotificationsYou must be signed in to change notification settings
Fork246
Star6k

[code-infra] Update changelog script to use utils from code-infra#2823

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

brijeshb42 merged 1 commit intomui:masterfrombrijeshb42:changelog-utils

Oct 10, 2025

Merged

[code-infra] Update changelog script to use utils from code-infra#2823

brijeshb42 merged 1 commit intomui:masterfrombrijeshb42:changelog-utils

Oct 10, 2025

Conversation

Copy link

Contributor

brijeshb42 commentedSep 24, 2025

I have followed (at least) thePR section of the contributing guide.

brijeshb42 requested review fromatomiks andmichaldudak ascode owners

September 24, 2025 13:09

brijeshb42 added the scope: code-infraInvolves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). label

Sep 24, 2025

Copy link

pkg-pr-newbot commentedSep 24, 2025•
edited
Loading

vite-css-base-ui-example

pnpm add https://pkg.pr.new/mui/base-ui/@base-ui-components/react@2823

pnpm add https://pkg.pr.new/mui/base-ui/@base-ui-components/utils@2823

commit:fbb2eb1

Copy link

mui-bot commentedSep 24, 2025•
edited
Loading

Bundle size report

Bundle	Parsed size	Gzip size
@base-ui-components/react	0B^(0.00%)	0B^(0.00%)

Details of bundle changes

Copy link

netlifybot commentedSep 24, 2025•
edited
Loading

✅ Deploy Preview forbase-ui ready!

Name	Link
🔨 Latest commit	`fbb2eb1`
🔍 Latest deploy log	https://app.netlify.com/projects/base-ui/deploys/68e7a84637b1980008287003
😎 Deploy Preview	https://deploy-preview-2823--base-ui.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to yourNetlify project configuration.

brijeshb42 mentioned this pull request

Sep 24, 2025

[code-infra] Add util to fetch changelog commits from githubmui/mui-public#742

Merged

1 task

brijeshb42 force-pushed thechangelog-utils branch from3eb64f3 to9e0218aCompare

September 26, 2025 09:27

github-actionsbot added the PR: out-of-dateThe pull request has merge conflicts and can't be merged. label

Sep 26, 2025

brijeshb42 force-pushed thechangelog-utils branch from9e0218a to3a3356dCompare

September 26, 2025 12:03

github-actionsbot removed the PR: out-of-dateThe pull request has merge conflicts and can't be merged. label

Sep 26, 2025

brijeshb42 force-pushed thechangelog-utils branch from3a3356d toa512d4bCompare

September 26, 2025 12:11

github-actionsbot added the PR: out-of-dateThe pull request has merge conflicts and can't be merged. label

Sep 29, 2025

brijeshb42 force-pushed thechangelog-utils branch froma512d4b tod1821d0Compare

October 1, 2025 12:27

github-actionsbot removed the PR: out-of-dateThe pull request has merge conflicts and can't be merged. label

Oct 1, 2025

brijeshb42 force-pushed thechangelog-utils branch fromd1821d0 to73c6a6bCompare

October 1, 2025 12:29

github-actionsbot added the PR: out-of-dateThe pull request has merge conflicts and can't be merged. label

Oct 1, 2025

brijeshb42 force-pushed thechangelog-utils branch from73c6a6b to982c089Compare

October 7, 2025 06:13

github-actionsbot removed the PR: out-of-dateThe pull request has merge conflicts and can't be merged. label

Oct 7, 2025

brijeshb42 force-pushed thechangelog-utils branch 2 times, most recently fromc95d68b to9e25f1aCompare

October 7, 2025 06:35

github-actionsbot added the PR: out-of-dateThe pull request has merge conflicts and can't be merged. label

Oct 8, 2025

brijeshb42 force-pushed thechangelog-utils branch from9e25f1a to8d27de6Compare

October 9, 2025 07:34

github-actionsbot removed the PR: out-of-dateThe pull request has merge conflicts and can't be merged. label

Oct 9, 2025

brijeshb42 requested a review froma team

October 9, 2025 07:35

brijeshb42 force-pushed thechangelog-utils branch from8d27de6 to88e29f3Compare

October 9, 2025 07:35

Janpot reviewed

Oct 9, 2025

View reviewed changes

scripts/releaseChangelog.mts OutdatedShow resolvedHide resolved

Janpot reviewed

Oct 9, 2025

View reviewed changes

scripts/releaseChangelog.mts

		request:{
		fetch,
		},
		constlatestTaggedVersion=awaitfindLatestTaggedVersion({

Copy link

Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Didn't realize before, but is there sufficient overlap betweenfindLatestTaggedVersion andgetLastCanaryTag to justify a ahared abstraction?

Copy link

ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Somewhat similar in that both fetches tags. ButgetLastCanaryTag fetches just that one tag whereas the otherone needs to fetch all to determine that latest one.
I don't see a reason to abstract them just yet.