Skip to content

update in labels: adding comment on the role of _coefname#895

Merged
s3alfisc merged 5 commits intopy-econometrics:masterfrom
mortizm1988:update_labels_doc
May 8, 2025
Merged

update in labels: adding comment on the role of _coefname#895
s3alfisc merged 5 commits intopy-econometrics:masterfrom
mortizm1988:update_labels_doc

Conversation

@mortizm1988
Copy link
Copy Markdown
Contributor

I modified the documentation on the three parts, as suggested by @s3alfisc

@codecov
Copy link
Copy Markdown

codecov bot commented May 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ
core-tests 80.35% <ø> (ø)
tests-extended ?
tests-vs-r 47.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
pyfixest/report/summarize.py 87.73% <ø> (ø)
pyfixest/report/visualize.py 77.45% <ø> (-4.63%) ⬇️

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@s3alfisc
Copy link
Copy Markdown
Member

s3alfisc commented May 8, 2025

pre-commit.ci autofix

@s3alfisc
Copy link
Copy Markdown
Member

s3alfisc commented May 8, 2025

@all-contributors please add @mortizm1988 for docs

@s3alfisc s3alfisc merged commit dd8e097 into py-econometrics:master May 8, 2025
1 check was pending
@allcontributors
Copy link
Copy Markdown
Contributor

@s3alfisc

I've put up a pull request to add @mortizm1988! 🎉

@s3alfisc
Copy link
Copy Markdown
Member

s3alfisc commented May 8, 2025

Perfect, thank you @mortizm1988!

damandhaliwal pushed a commit to damandhaliwal/pyfixest that referenced this pull request Jun 17, 2025
…trics#895)

* update in labels: adding comment on the role of _coefname

* fix typo

* fix typo

* fix typo

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: Alexander Fischer <alexander-fischer1801@t-online.de>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants