Skip to content

ci: Use custom action to commit changes in CI instead of git commit#792

Open
fnesveda wants to merge 1 commit into
masterfrom
chore/use-custom-action-to-commit
Open

ci: Use custom action to commit changes in CI instead of git commit#792
fnesveda wants to merge 1 commit into
masterfrom
chore/use-custom-action-to-commit

Conversation

@fnesveda
Copy link
Copy Markdown
Member

@fnesveda fnesveda commented May 14, 2026

We want to enforce commit signing for all commits in our repositories.
To do that, we need to make sure even commits created by CI workflows are signed.

It would be possible to sign using GPG keys, but that would require a lot of maintenance.

Instead, we can commit using the GitHub GraphQL API, which automatically signs commits.

This PR replaces direct git commit / git push usage (and third-party commit actions like EndBug/add-and-commit)
with the apify/workflows/commit action, which uses the GraphQL API under the hood.

As agreed on Slack, I'm also simplifying the workflows to always work on top of the default branch, rather than on a ref passed down through the input, as it should effectively be the same.

@github-actions github-actions Bot added the t-management Issues with this label belong to the CTO or engineering managers. label May 14, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.73%. Comparing base (d1bd5a3) to head (5291c8b).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #792   +/-   ##
=======================================
  Coverage   97.73%   97.73%           
=======================================
  Files          48       48           
  Lines        4860     4860           
=======================================
  Hits         4750     4750           
  Misses        110      110           
Flag Coverage Δ
integration 93.66% <ø> (ø)
unit 86.19% <ø> (ø)

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.

@fnesveda fnesveda added the adhoc Ad-hoc unplanned task added during the sprint. label May 14, 2026
@fnesveda fnesveda requested review from Pijukatel and vdusek May 14, 2026 09:46
@vdusek vdusek requested review from janbuchar and removed request for Pijukatel May 14, 2026 10:10
@vdusek vdusek changed the title chore: Use custom action to commit changes in CI instead of git commit ci: Use custom action to commit changes in CI instead of git commit May 14, 2026
Copy link
Copy Markdown
Contributor

@vdusek vdusek left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-management Issues with this label belong to the CTO or engineering managers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants