Skip to content

Conversation

@EZoni
Copy link
Member

@EZoni EZoni commented Nov 12, 2025

The CI workflow file was called unit_tests.yml and the job name was "unit tests". I propose that we rename the workflow file tests.yml and the job name "tests", since the workflow runs both unit tests and examples - not just unit tests. I would be open also to calling the workflow file tests_and_examples.yml and the job name "tests and examples".

@EZoni EZoni requested a review from ax3l November 12, 2025 16:33
@EZoni EZoni added the test label Nov 12, 2025
@EZoni
Copy link
Member Author

EZoni commented Nov 12, 2025

I can rename the workflow name itself too, as @ax3l had suggested in #3 (comment).

@EZoni EZoni force-pushed the rename_ci_workflow branch from dfa8739 to 6c1c886 Compare November 12, 2025 16:39
Copy link
Member

@ax3l ax3l left a comment

Choose a reason for hiding this comment

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

Thank you! :)

@ax3l ax3l merged commit 35ed73a into pals-project:main Dec 1, 2025
5 checks passed
@ax3l ax3l self-assigned this Dec 1, 2025
@EZoni EZoni deleted the rename_ci_workflow branch December 17, 2025 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants