Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Add Catch2 dependency and manage its installation with Taskfile workflows. #25

Open
wants to merge 30 commits into
base: main
Choose a base branch
from

Conversation

Bill-hbrhbr
Copy link
Collaborator

@Bill-hbrhbr Bill-hbrhbr commented Feb 11, 2025

Description

  • Download and install Catch2 dependency with the newest cmake and remote tasks in yscope-dev-utils.
  • Replace the current cmake config and build tasks in ystdlib-cpp with the newest versions in yscope-dev-utils.
  • Change the dummy target to a Catch2 unit test target.
  • Ensure all tasks have specified the correct sources, generates, and status to preserve necessary re-runs and prevent unnecessary runs.
  • Remove submodules as a potential folder to exclude when linting yaml files in ystdlib-cpp. Now every non-project file resides in build except for yscope-dev-utils.
  • Update the lint:venv task to comply with the arglist of new checksum tasks in yscope-dev-utils.

Most importantly: the highest-order-bit task build:all should trigger a re-run whenever there's a dependency change.

Checklist

  • The PR satisfies the contribution guidelines.
  • This is a breaking change and that has been indicated in the PR title, OR this isn't a
    breaking change.
  • Necessary docs have been updated, OR no docs need to be updated.

Validation performed

  • The new unit test target builds and runs succcessfully.

@Bill-hbrhbr Bill-hbrhbr changed the title chore: Task dep chore: Add Catch2 dependency and manage its installation with Taskfile workflows. Feb 12, 2025
@Bill-hbrhbr Bill-hbrhbr marked this pull request as ready for review February 12, 2025 00:50
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.

1 participant