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

Unit Testing Pt. I #80

Merged
merged 21 commits into from
Feb 22, 2024
Merged

Unit Testing Pt. I #80

merged 21 commits into from
Feb 22, 2024

Conversation

cbhernan
Copy link
Collaborator

@cbhernan cbhernan commented Feb 8, 2024

What does this PR do?

Groundwork for Unit Testing and CodeCov

Copy link

codecov bot commented Feb 9, 2024

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

@cbhernan cbhernan self-assigned this Feb 13, 2024
@cbhernan cbhernan marked this pull request as ready for review February 21, 2024 17:52
@cbhernan cbhernan requested a review from a team as a code owner February 21, 2024 17:52
@cbhernan cbhernan changed the title [WIP] Unit Testing Unit Testing Pt. I Feb 21, 2024
Copy link

github-actions bot commented Feb 21, 2024

Visit the preview URL for this PR (updated for commit 1d2385c):

https://angeleno-my-account-dev-402917--pr80-unit-testing-wwlxe82m.web.app

(expires Thu, 29 Feb 2024 18:48:41 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: ef7f92b444e46d3ebe3d59054b9a245afee849f5

Copy link
Contributor

@nickchaula nickchaula left a comment

Choose a reason for hiding this comment

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

Looks good. Approved

@cbhernan cbhernan merged commit 457b25d into development Feb 22, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants