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

Update pylint version in setup.py #393

Closed
wants to merge 1 commit into from

Conversation

sweep-ai[bot]
Copy link

@sweep-ai sweep-ai bot commented Aug 23, 2023

Description

This PR updates the pylint version in the tests list of the setup.py file. The current version is less than 2.14, and this PR updates it to the latest version.

Summary of Changes

  • Updated the pylint version in the tests list of the setup.py file from <2.14 to >=2.14.

Please review and merge at your earliest convenience.

Fixes #389.


To checkout this PR branch, run the following command in your terminal:

git checkout {pull_request.branch_name}

🎉 Latest improvements to Sweep:

  • Use Sweep Map to break large issues into smaller sub-issues, perfect for large tasks like "Sweep (map): migrate from React class components to function components"
  • Getting Sweep to format before committing! Check out Sweep Sandbox Configs to set it up.
  • We released a demo of our chunker, where you can find the corresponding blog and code.

💡 To get Sweep to edit this pull request, you can:

  • Leave a comment below to get Sweep to edit the entire PR
  • Leave a comment in the code will only modify the file
  • Edit the original issue to get Sweep to recreate the PR from scratch

@sweep-ai sweep-ai bot added the sweep Assigns Sweep to an issue or pull request. label Aug 23, 2023
@sweep-ai sweep-ai bot mentioned this pull request Aug 23, 2023
1 task
@sweep-ai
Copy link
Author

sweep-ai bot commented Aug 23, 2023

GitHub actions yielded the following error.

The relevant lines from the logs for the failed command "pre-commit run pylint -a -v --show-diff-on-failure" are:

Using config file /home/runner/work/gto/gto/.pylintrc
************* Module /home/runner/work/gto/gto/.pylintrc
.pylintrc:1:0: R0022: Useless option value for '--disable', 'print-statement' was removed from pylint, see https://github.com/pylint-dev/pylint/pull/4942. (useless-option-value)

And for the failed command "pre-commit run pylint -a -v --show-diff-on-failure" on Windows:

Using config file /home/runner/work/gto/gto/.pylintrc
************* Module D:\a\gto\gto\.pylintrc
.pylintrc:1:0: R0022: Useless option value for '--disable', 'print-statement' was removed from pylint, see https://github.com/pylint-dev/pylint/pull/4942. (useless-option-value)

These lines indicate that there is a useless option value for '--disable' in the .pylintrc file.

There are a lot of errors. This is likely due to a small parsing issue or a missing import with the files changed in the PR.

This is likely a linting or type-checking issue with the source code. Update the code changed by the PR. Don't modify the existing tests.

I'm getting the same errors 3 times in a row, so I will stop working on fixing this PR.

@aguschin aguschin closed this Aug 24, 2023
@aguschin aguschin deleted the sweep/update-pylint-version branch August 24, 2023 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sweep Assigns Sweep to an issue or pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sweep: Update pylint version in setup.py
1 participant