Skip to content

Conversation

@antonwolfy
Copy link
Contributor

@antonwolfy antonwolfy commented Feb 17, 2024

The PR extends pre-commit hooks with pylint configuration.
The current files filter for the verification is set to dpnp/dpnp_iface*.py (excluding files from #1712) to reduce number of issues. The other files will be added to the scope in later PRs

  • Have you provided a meaningful PR description?
  • Have you added a test, reproducer or referred to issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • Have you checked performance impact of proposed changes?
  • If this PR is a work in progress, are you filing the PR as a draft?

@antonwolfy antonwolfy self-assigned this Feb 17, 2024
@antonwolfy antonwolfy marked this pull request as draft February 17, 2024 16:04
@github-actions
Copy link
Contributor

github-actions bot commented Feb 17, 2024

View rendered docs @ https://intelpython.github.io/dpnp/pull//index.html

@antonwolfy antonwolfy force-pushed the add_pylint_to_pre_commit branch from 349d86c to 27c9e3c Compare February 19, 2024 09:24
antonwolfy and others added 2 commits February 19, 2024 21:47
…ce_searching.py dpnp_iface_sorting.py dpnp_iface_statistics.py dpnp_iface_types.py
@antonwolfy antonwolfy marked this pull request as ready for review February 19, 2024 20:49
@antonwolfy antonwolfy requested review from npolina4, vlad-perevezentsev and vtavana and removed request for npolina4 and vtavana February 19, 2024 20:49
Copy link
Contributor

@vlad-perevezentsev vlad-perevezentsev left a comment

Choose a reason for hiding this comment

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

LGTM!
Thank you @antonwolfy for adding a useful pre-commit hook

@antonwolfy antonwolfy merged commit e6e7b3e into master Feb 20, 2024
@antonwolfy antonwolfy deleted the add_pylint_to_pre_commit branch February 20, 2024 14:25
github-actions bot added a commit to antonwolfy/dpnp that referenced this pull request Feb 20, 2024
* Resolved all issue in dpnp_iface.py

* Make pylint as a local hook

* Added dpnp_iface_arraycreation.py to the pylint hook

* Added dpnp_iface_counting.py and dpnp_iface_indexing.py

* Added dpnp_iface_libmath.py and dpnp_iface_linearalgebra.py

* Adding dpnp_iface_manipulation.py dpnp_iface_nanfunctions.py dpnp_iface_searching.py dpnp_iface_sorting.py dpnp_iface_statistics.py dpnp_iface_types.py

* Disable D205 flake8 warning globally e6e7b3e
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