-
Notifications
You must be signed in to change notification settings - Fork 395
Add new gemfiles to lock-dependency commit #4962
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
base: master
Are you sure you want to change the base?
Conversation
Typing analysisIgnored filesThere are 539 ignored files in the Steepfile out of 836. Ignored files
Note: Ignored files are excluded from the next sections.
|
✅ Tests 🎉 All green!❄️ No new flaky tests detected 🎯 Code Coverage 🔗 Commit SHA: 4828852 | Docs | Was this helpful? Give us feedback! |
BenchmarksBenchmark execution time: 2025-10-08 21:43:54 Comparing candidate commit 4828852 in PR branch Found 1 performance improvements and 0 performance regressions! Performance is the same for 43 metrics, 2 unstable metrics. scenario:tracing - Propagation - Datadog
|
path: gemfiles | ||
pattern: lock-dependency-${{ github.run_id }}-* | ||
merge-multiple: true | ||
- name: Stage changes # Since ghcommit-action does not pick up unstaged files |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should run after git diff
, otherwise you need to adjust diff to show staged changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
git diff
will not show staged files, one solution should be git add -u
after git add gemfiles
and git diff --cached
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think a simple order change would do a thing
This issue was noticed recently (#4924 (comment)) and it caused by a known limitation of planetscale/ghcommit-action.
Here you can see where our job failed to pick up the newly added gemfile.lock: https://github.com/DataDog/dd-trace-rb/actions/runs/18348438937/job/52262026349#step:7:298
Change log entry
No