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

Deps updates #8207

Merged
merged 7 commits into from
Apr 8, 2025
Merged

Deps updates #8207

merged 7 commits into from
Apr 8, 2025

Conversation

wraithgar
Copy link
Member

@wraithgar wraithgar requested a review from a team as a code owner April 3, 2025 16:19
@owlstronaut owlstronaut force-pushed the gar/deps-updates branch 4 times, most recently from a577463 to 92e3cf5 Compare April 3, 2025 21:00
@Joe1the2Creator3

This comment was marked as spam.

@owlstronaut
Copy link
Contributor

owlstronaut commented Apr 4, 2025

Okay! I was able to validate that the change of 41/42 packages installed was in fact from the optional dependency change (failing in latest). I'm surprised the smoke test didn't run against that PR? In any case, this is the after and before of the logs:

image

These exactly account for the number change, so the number change is correct ( I counted the number of silly ADD with no value after )

AKayy2020

This comment was marked as spam.

@santanderjohnbruce9

This comment was marked as spam.

@wraithgar wraithgar merged commit 7fb624f into latest Apr 8, 2025
65 checks passed
@wraithgar wraithgar deleted the gar/deps-updates branch April 8, 2025 02:21
@github-actions github-actions bot mentioned this pull request Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants