Skip to content

Conversation

@insop
Copy link

@insop insop commented Feb 5, 2026

Using a default value for USER prevents failures in environments where it’s unset (e.g., many containers). This change worked for me, and I can open a PR to apply it to other config files if desired.

  • I have personally reviewed this PR and description before asking others to do so. It meets the quality bar I expect from others. I understand that if this PR is perceived as unverified AI-generated code, it will be closed without further explanation.
  • I have run tests and confirmed that this code works. I ran this code for apps/grpo

Description

Addresses #742

Using a default value for USER prevents failures in environments where it’s unset (e.g., many containers). This change worked for me, and I can open a PR to apply it to other config files if desired.

Test plan

  • I ran apps/grpo with this code.
  • pre-commit run --all-files ran clean

Using a default value for USER prevents failures in environments where it’s unset (e.g., many containers). This change worked for me, and I can open a PR to apply it to other config files if desired.
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Feb 5, 2026
@insop
Copy link
Author

insop commented Feb 5, 2026

@daniellepintz , PTAL.

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

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant