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

feat: added --ignore-init-modules flag (fixes #226) #228

Merged
merged 4 commits into from
Oct 14, 2022

Conversation

ErikBjare
Copy link
Contributor

@ErikBjare ErikBjare commented Oct 13, 2022

fixes #226

Checklist

  • Add test cases to all the changes you introduce
    • Change deemed trivial, testing seems overkill
  • Update the documentation for the changes

@ErikBjare ErikBjare force-pushed the dev/ignore-init-modules branch from 9244eac to f6d78cf Compare October 13, 2022 11:13
@ErikBjare ErikBjare changed the title feat: added --ignore-init-module-imports flag (fixes #226) feat: added --ignore-init-module flag (fixes #226) Oct 13, 2022
@ErikBjare ErikBjare force-pushed the dev/ignore-init-modules branch from f6d78cf to b0ab496 Compare October 13, 2022 11:15
@ErikBjare ErikBjare changed the title feat: added --ignore-init-module flag (fixes #226) feat: added --ignore-init-modules flag (fixes #226) Oct 13, 2022
@coveralls
Copy link

coveralls commented Oct 13, 2022

Pull Request Test Coverage Report for Build 3241939687

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 3 of 4 (75.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.3%) to 95.975%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/autoimport/entrypoints/cli.py 3 4 75.0%
Totals Coverage Status
Change from base Build 3069979301: -0.3%
Covered Lines: 310
Relevant Lines: 323

💛 - Coveralls

Copy link
Owner

@lyz-code lyz-code left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution @ErikBjare

@lyz-code lyz-code merged commit 1dc6138 into lyz-code:main Oct 14, 2022
@ErikBjare
Copy link
Contributor Author

Thank YOU for an awesome tool! ❤️

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.

Flag to skip for __init__.py files
3 participants