Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

test: add CTest to cmake to enable testing#2753

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
realstealthninja wants to merge4 commits intoTheAlgorithms:master
base:master
Choose a base branch
Loading
fromrealstealthninja:ctest

Conversation

realstealthninja
Copy link
Collaborator

@realstealthninjarealstealthninja commentedOct 4, 2024
edited
Loading

Description of Change

Testing

This pr enables testing of the algorithms in the repository
before we get to the stage of running tests on a CI. We will need to remove all
other code from every main function so that the only remaining code is a call to the test function.

for now this draft pull request shows how easy it is to enable testing.
adding it to the awesome CI will be as easy as adding
make test to a test step.

Checklist

  • Added description of change
  • Added file name matchesFile name guidelines
  • Added tests and example, test must pass
  • Added documentation so that the program is self-explanatory and educational -Doxygen guidelines
  • Relevant documentation/comments is changed or added
  • PR title follows semanticcommit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:

@github-actionsGitHub Actions
Copy link
Contributor

This pull request has been automatically marked as abandoned because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actionsgithub-actionsbot added the staleAuthor has not responded to the comments for over 2 weeks labelNov 4, 2024
@realstealthninjarealstealthninja added dont-closeThis issue/pull request shouldn't be closed and removed staleAuthor has not responded to the comments for over 2 weeks labelsNov 4, 2024
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@Panquesito7Panquesito7Awaiting requested review from Panquesito7Panquesito7 will be requested when the pull request is marked ready for reviewPanquesito7 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
dont-closeThis issue/pull request shouldn't be closedtesting
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

1 participant
@realstealthninja

[8]ページ先頭

©2009-2025 Movatter.jp