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

Update ci.yml #75

Merged
merged 1 commit into from
Oct 8, 2023
Merged

Conversation

ayushawasthi24
Copy link
Contributor

Pull Request for GitHub Actions Workflow

Description

In this pull request, I've made changes to the GitHub Actions workflow file (ci.yml). The goal of these changes is to automate the detection and execution of all test files in the tests/ directory, making the workflow more efficient and user-friendly.

Changes Made

  • Modified the workflow file ci.yml to dynamically discover all test files in the tests/ directory.
  • Updated the workflow steps to run pytest on all discovered test files.

Testing

I've tested these changes locally to ensure they work as intended. I ran the modified workflow file in a similar repository and confirmed that it automatically detected and executed all test files in the tests/ directory.

Copy link
Member

@audreyfeldroy audreyfeldroy left a comment

Choose a reason for hiding this comment

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

Amazing, thanks for contributing this @ayushawasthi24! Approved and merging 🚢

@audreyfeldroy
Copy link
Member

Test failure here is related to the CLI version perimeter test, which is beyond the scope of this issue. Continuing with merge and will address that separately. 🙂

@audreyfeldroy audreyfeldroy merged commit fe20980 into crisishistory:main Oct 8, 2023
1 check failed
@ayushawasthi24
Copy link
Contributor Author

Thankyou!! Hoping to contribute more.

@audreyfeldroy audreyfeldroy added the hacktoberfest-accepted Issue or PR is approved for anyone who wants it to count toward Hacktoberfest label Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Issue or PR is approved for anyone who wants it to count toward Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants