Skip to content

Conversation

ckohen
Copy link
Member

@ckohen ckohen commented Aug 11, 2025

Please describe the changes this PR makes and why it should be merged:

Still need to set a user...since we have an app now we'll use that instead

Status and versioning classification:

Copy link

vercel bot commented Aug 11, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
discord-js Ignored Ignored Preview Aug 16, 2025 9:03pm
discord-js-guide Ignored Ignored Preview Aug 16, 2025 9:03pm

@Jiralite Jiralite added the ci label Aug 11, 2025
vladfrangu
vladfrangu previously approved these changes Aug 11, 2025
almeidx
almeidx previously approved these changes Aug 11, 2025
@ckohen ckohen requested a review from a team as a code owner August 12, 2025 01:09
@ckohen ckohen requested a review from Jiralite August 12, 2025 01:09
Copy link

codecov bot commented Aug 12, 2025

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 44.06%. Comparing base (bdcb3e1) to head (b4e48ba).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...ages/actions/src/releasePackages/releasePackage.ts 0.00% 3 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main   #11038   +/-   ##
=======================================
  Coverage   44.06%   44.06%           
=======================================
  Files         310      310           
  Lines       17755    17755           
  Branches     1755     1755           
=======================================
  Hits         7824     7824           
  Misses       9919     9919           
  Partials       12       12           
Flag Coverage Δ
utilities 6.32% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ckohen ckohen changed the title ci(release): set git user ci(release): use app user Aug 12, 2025
@ckohen ckohen requested a review from vladfrangu August 12, 2025 04:13
@almeidx almeidx dismissed stale reviews from vladfrangu and themself August 12, 2025 11:31

Stale

@github-project-automation github-project-automation bot moved this from Todo to Review Approved in discord.js Aug 16, 2025
@kodiakhq kodiakhq bot merged commit ac290ae into discordjs:main Aug 16, 2025
7 checks passed
@github-project-automation github-project-automation bot moved this from Review Approved to Done in discord.js Aug 16, 2025
@ckohen ckohen deleted the release/set-git-user branch August 18, 2025 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants