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

Adding pylint hooks for dpbench #332

Closed
wants to merge 2 commits into from
Closed

Conversation

adarshyoga
Copy link
Contributor

@adarshyoga adarshyoga commented Feb 7, 2024

  • Have you provided a meaningful PR description?

This PR replicates the pylint hooks from numba-dpex repo to dpbench.
TODO: Address pylint warnings and get pre-commit to pass.

  • Have you added a test, reproducer or referred to an 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?
  • If this PR is a work in progress, are you filing the PR as a draft?

@adarshyoga adarshyoga requested a review from ZzEeKkAa February 7, 2024 22:05
@adarshyoga adarshyoga self-assigned this Feb 7, 2024
@adarshyoga adarshyoga force-pushed the sw_feature/add_pylint branch from 14d366f to 718f22b Compare February 7, 2024 22:05
@adarshyoga adarshyoga marked this pull request as draft February 7, 2024 22:44
@adarshyoga adarshyoga force-pushed the sw_feature/add_pylint branch 3 times, most recently from f982f88 to 940f870 Compare February 14, 2024 06:25
@adarshyoga adarshyoga force-pushed the sw_feature/add_pylint branch 6 times, most recently from 52f4be8 to 089fc0b Compare February 23, 2024 20:49
@adarshyoga adarshyoga changed the base branch from main to fix/directory_structure February 23, 2024 20:49
Base automatically changed from fix/directory_structure to main February 23, 2024 21:13
@adarshyoga adarshyoga force-pushed the sw_feature/add_pylint branch from 6f6219b to 6215cf8 Compare February 26, 2024 20:33
@adarshyoga adarshyoga closed this Apr 26, 2024
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.

1 participant