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

Refactor codebase to apply selected ruff linting and formatting rules… #312

Merged
merged 2 commits into from
Feb 13, 2024

Conversation

s3alfisc
Copy link
Member

@s3alfisc s3alfisc commented Feb 13, 2024

Refactor codebase to apply selected ruff linting and formatting rules.

NKeleher and others added 2 commits February 13, 2024 19:32
…#311)

* add ruff linting and formatting rules

* basic ruff autofix

* update pyproject

* refactor utils.py

* refactoring modules for ruff linting

* Add ruff linting formatting and basic ruff autofixes (#305)

* add ruff linting and formatting rules

* basic ruff autofix

* lint / black

* bump version

---------

Co-authored-by: Niall Keleher <[email protected]>

* update readme (#308)

* update readme

* update readme

* fix typo

* more readme udpates

* update pyfixest.qmd

* only return md table in readme

* dont return rwolf output in readme

* attempt to print data frame

* add examples for rwolf

* add examples for rwolf

* fix small typos

* add bonferroni correction #309 (#310)

* add bonferroni correction #309

* Update rwolf and bonferroni docs

* update version to 0.15.4

* basic ruff autofix

* order imports

* basic ruff autofix

* Add ruff linting formatting and basic ruff autofixes (#305)

* add ruff linting and formatting rules

* basic ruff autofix

* lint / black

* bump version

---------

Co-authored-by: Niall Keleher <[email protected]>

* add black pre-commit hook

* refactoring modules for ruff linting

* add ruff to linting github action

* disable ruff on notebooks

* format FormulaParser

* temp - run tests on fix-linting-errors

* fix dataframe filtering, noqa: E712

* fix dropped_singleton_bool conditional

* remove extra debugging branch on CI

---------

Co-authored-by: Alexander Fischer <[email protected]>
@s3alfisc s3alfisc merged commit 5408f25 into master Feb 13, 2024
8 checks passed
@s3alfisc s3alfisc deleted the fix-linting-errors branch February 13, 2024 18:57
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