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.

Sign up for GitHub

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

Jump to bottom

Created tests for the existing math modules #31

Merged
merged 2 commits into from
Oct 12, 2024
Merged

Created tests for the existing math modules #31

merged 2 commits into from
Oct 12, 2024

Conversation

Ramy-Badr-Ahmed
Copy link
Member

@Ramy-Badr-Ahmed Ramy-Badr-Ahmed commented Oct 9, 2024 •
edited
Loading

Contents

  • Created tests to simulate unit testing.
  • Added new test files:
    • euclid.f90: Contains unit tests for the gcd_module.
    • factorial.f90: Contains unit tests for the factorial_module.
    • fibonacci.f90: Contains unit tests for the fibonacci_module.

Note

Addendum to my PRs: #15, #16, #17

Ramy-Badr-Ahmed
Ramy-Badr-Ahmed commented Oct 9, 2024
View reviewed changes
Copy link
Member Author

@Ramy-Badr-Ahmed Ramy-Badr-Ahmed left a comment

Choose a reason for hiding this comment

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

Hi @SatinWukerORIG ,
Looking forward to your review 🙂

ci

check_code_format

SatinWukerORIG
SatinWukerORIG approved these changes Oct 12, 2024
View reviewed changes
Copy link
Member

@SatinWukerORIG SatinWukerORIG left a comment

Choose a reason for hiding this comment

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

Sorry for the delayed review!

@SatinWukerORIG SatinWukerORIG merged commit 4e8de95 into TheAlgorithms:main Oct 12, 2024
3 checks passed
@Ramy-Badr-Ahmed
Copy link
Member Author

Sorry for the delayed review!

Thanks @SatinWukerORIG!
No problem 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SatinWukerORIG SatinWukerORIG SatinWukerORIG approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Ramy-Badr-Ahmed @SatinWukerORIG