Skip to content
This repository was archived by the owner on Nov 19, 2025. It is now read-only.

ci: Use step-security changed-files action - #536

Open
chtruong814 wants to merge 1 commit into
mainfrom
step-security-changed-files
Open

ci: Use step-security changed-files action#536
chtruong814 wants to merge 1 commit into
mainfrom
step-security-changed-files

Conversation

@chtruong814

Copy link
Copy Markdown
Collaborator

What does this PR do ?

Use step-security changed-files action. They made it available to everyone as the more secure drop-in replacement.

Changelog

  • Use step-security changed-files action

Before your PR is "Ready for review"

Pre checks:

Checklist when contributing a new algorithm

  • Does the trainer resume and restore model state all states?
  • Does the trainer support all parallelism techniques(PP, TP, DP)?
  • Does the trainer support max_steps=-1 and validation?
  • Does the trainer only call APIs defined in alignable_interface.py?
  • Does the trainer have proper logging?

Additional Information

  • Related to # (issue)

@chtruong814 chtruong814 added the Run CICD Set + un-set to retrigger (add after r*.*.* labels) label Mar 19, 2025
@github-actions github-actions Bot added the CI label Mar 19, 2025
@chtruong814 chtruong814 changed the title Use step-security changed-files action ci: Use step-security changed-files action Mar 19, 2025
ko3n1g
ko3n1g previously approved these changes Mar 19, 2025
@ko3n1g ko3n1g added Skip CICD and removed Run CICD Set + un-set to retrigger (add after r*.*.* labels) labels Mar 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants