Skip to content

Exclude *.test.cjs files from sync and runtime file copy#30

Merged
pelikhan merged 2 commits intomainfrom
copilot/update-actions-to-ignore-tests
Mar 19, 2026
Merged

Exclude *.test.cjs files from sync and runtime file copy#30
pelikhan merged 2 commits intomainfrom
copilot/update-actions-to-ignore-tests

Conversation

Copy link
Contributor

Copilot AI commented Mar 19, 2026

220 JavaScript test files were being synced from gh-aw into this repo and then skipped at runtime — unnecessary noise committed to the repo.

Changes

  • sync-actions.yml: Added --exclude='*.test.cjs' to the rsync command so future syncs don't pull test files into the repo
  • setup/js/: Removed all 220 existing *.test.cjs files that had been committed via previous syncs
# Before
rsync --archive --verbose --delete --exclude='README*' "gh-aw/actions/$name/" "gh-aw-actions/$name/"

# After
rsync --archive --verbose --delete --exclude='README*' --exclude='*.test.cjs' "gh-aw/actions/$name/" "gh-aw-actions/$name/"

Note: setup.sh already had runtime skip logic for *.test.cjs (lines 105–109); the gap was upstream at the sync step.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan marked this pull request as ready for review March 19, 2026 00:21
Copilot AI review requested due to automatic review settings March 19, 2026 00:21
Copy link
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

This PR reduces repo noise by preventing *.test.cjs files from being synced in from gh-aw and deletes previously committed synced test files that were already skipped at runtime.

Changes:

  • Update the rsync command in sync-actions.yml to exclude *.test.cjs.
  • Remove the previously synced setup/js/*.test.cjs files from the repository.

Reviewed changes

Copilot reviewed 49 out of 221 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/workflows/sync-actions.yml Excludes *.test.cjs during rsync to prevent future sync churn
setup/js/create_discussion_fallback.test.cjs Removes previously synced test file
setup/js/create_discussion_category_normalization.test.cjs Removes previously synced test file
setup/js/create_code_scanning_alert.test.cjs Removes previously synced test file
setup/js/create_agent_session.test.cjs Removes previously synced test file
setup/js/constants.test.cjs Removes previously synced test file
setup/js/comment_limit_helpers.test.cjs Removes previously synced test file
setup/js/close_pull_request.test.cjs Removes previously synced test file
setup/js/close_older_entities.test.cjs Removes previously synced test file
setup/js/close_expired_pull_requests.test.cjs Removes previously synced test file
setup/js/close_expired_issues.test.cjs Removes previously synced test file
setup/js/close_expired_discussions.test.cjs Removes previously synced test file
setup/js/close_entity_helpers.test.cjs Removes previously synced test file
setup/js/close_discussion.test.cjs Removes previously synced test file
setup/js/check_workflow_timestamp.test.cjs Removes previously synced test file
setup/js/check_workflow_recompile_needed.test.cjs Removes previously synced test file
setup/js/check_team_member.test.cjs Removes previously synced test file
setup/js/check_stop_time.test.cjs Removes previously synced test file
setup/js/check_skip_roles.test.cjs Removes previously synced test file
setup/js/check_skip_if_no_match.test.cjs Removes previously synced test file
setup/js/check_skip_if_match.test.cjs Removes previously synced test file
setup/js/check_skip_bots.test.cjs Removes previously synced test file
setup/js/check_permissions.test.cjs Removes previously synced test file
setup/js/check_command_position.test.cjs Removes previously synced test file
setup/js/check_admin_permissions.test.cjs Removes previously synced test file
setup/js/call_workflow.test.cjs Removes previously synced test file
setup/js/autofix_code_scanning_alert.test.cjs Removes previously synced test file
setup/js/assign_milestone.test.cjs Removes previously synced test file
setup/js/assign_issue.test.cjs Removes previously synced test file
setup/js/assign_copilot_to_created_issues.test.cjs Removes previously synced test file
setup/js/add_workflow_run_comment.test.cjs Removes previously synced test file
setup/js/add_reviewer.test.cjs Removes previously synced test file
setup/js/add_reaction.test.cjs Removes previously synced test file
setup/js/add_copilot_reviewer.test.cjs Removes previously synced test file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@pelikhan pelikhan merged commit ccf2cf9 into main Mar 19, 2026
5 checks passed
@pelikhan pelikhan deleted the copilot/update-actions-to-ignore-tests branch March 19, 2026 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants