Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Codecov #628

Merged
merged 6 commits into from
Sep 22, 2024
Merged

Codecov #628

merged 6 commits into from
Sep 22, 2024

Conversation

s3alfisc
Copy link
Member

@s3alfisc s3alfisc commented Sep 21, 2024 •
edited
Loading

Add some tests for event_study() function.

@codecov Codecov
Copy link

codecov bot commented Sep 21, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 39.65517% with 35 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
tests/test_event_study.py 36.36% 35 Missing ⚠️
Files with missing lines Coverage Δ
pyfixest/did/estimation.py 96.49% <ø> (+10.28%) ⬆️
pyfixest/estimation/feols_.py 85.36% <100.00%> (+3.24%) ⬆️
tests/test_event_study.py 36.36% <36.36%> (ø)

... and 35 files with indirect coverage changes

@s3alfisc s3alfisc merged commit 5bf8cc9 into master Sep 22, 2024
9 of 10 checks passed
@s3alfisc s3alfisc deleted the codecov branch September 22, 2024 15:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@s3alfisc