Skip to content

switch user in crowdin action instead of changing owner after run#22554

Open
toy wants to merge 1 commit intodevfrom
update-crowdin-action-to-use-user-input
Open

switch user in crowdin action instead of changing owner after run#22554
toy wants to merge 1 commit intodevfrom
update-crowdin-action-to-use-user-input

Conversation

@toy
Copy link
Copy Markdown
Contributor

@toy toy commented Mar 27, 2026

Ticket

What are you trying to accomplish?

The Crowdin GitHub Action runs as root inside a Docker container, causing downloaded translation files to be owned by root. This breaks subsequent workflow steps that need to edit or git add those files.

What approach did you choose and why?

Use the user: auto input added in crowdin/github-action#309 (shipped in v2.16.0) to run the action as the workspace owner instead of root. This avoids the file ownership problem at the source, removing the need for the follow-up sudo chown step.

  • Added user: auto to the crowdin/github-action step so the Docker container runs as the workspace owner (detected from .git directory ownership).
  • Removed the "Fix ownership of downloaded translation files" step that previously ran sudo chown to correct ownership after the fact.

Merge checklist

  • Added/updated tests
  • Added/updated documentation in Lookbook (patterns, previews, etc)
  • Tested major browsers (Chrome, Firefox, Edge, ...)

📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Crowdin synchronization GitHub workflow to avoid post-run file ownership fixes by attempting to run the Crowdin action under a non-root user.

Changes:

  • Adds user: auto input to the crowdin/github-action step.
  • Removes the follow-up sudo chown step that previously corrected ownership of downloaded translation files.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants