Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): run yarn upgrade #38022

Merged
merged 1 commit into from
Feb 9, 2025
Merged

chore(deps): run yarn upgrade #38022

merged 1 commit into from
Feb 9, 2025

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Feb 7, 2025

Description

Ran yarn upgrade to update all indirect dependencies.

Motivation

mdn/content is using some outdated indirect dependencies.

Additional details

Related issues and pull requests

Previous:

@caugner caugner requested a review from mdn-bot as a code owner February 7, 2025 11:07
@github-actions github-actions bot added system [PR only] Infrastructure and configuration for the project size/xl [PR only] >1000 LoC changed labels Feb 7, 2025
@caugner caugner requested a review from bsmth February 7, 2025 11:08
Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

ty!

@Josh-Cena Josh-Cena merged commit 5368a5a into main Feb 9, 2025
16 of 17 checks passed
@Josh-Cena Josh-Cena deleted the yarn-upgrade branch February 9, 2025 16:52
@ddbeck
Copy link
Contributor

ddbeck commented Feb 10, 2025

@caugner FYI, this commit has broken generating the content inventory. When running yarn --silent run content inventory, commits before this work fine but commits after this emit some text about "Updating @webref/css" before the inventory. See https://github.com/ddbeck/mdn-content-inventory/actions/runs/13249294934/job/36983168806#step:5:24.

I haven't been able to reproduce this locally yet, so I'm not 100% certain what the problem is. I'll look into it more tomorrow, but if you have any ideas here I'd be curious to know what you think.

@caugner
Copy link
Contributor Author

caugner commented Feb 17, 2025

@ddbeck I opened ddbeck/mdn-content-inventory#7 to mitigate the issue using the recommended workaround.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/xl [PR only] >1000 LoC changed system [PR only] Infrastructure and configuration for the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants