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

Pylint - Run Against Current Subtree If No External Files Found #3140

Merged
merged 5 commits into from Feb 10, 2023

Conversation

tbpassin
Copy link
Contributor

@tbpassin tbpassin commented Feb 9, 2023

Now searches for external files up and down from the current node. If found, runs pylint on all external Python files found. If not, runs pylint on the tree headed by the current node.

To do this we have to write the subtree to a temp file, carry the target position through to printing the line to the log pane and use it to compute the right UNL for navigation.
Now searches for external files up and down from the current node.  If found, runs pylint on all external Python files found.  If not, runs pylint on the tree headed by the current node.  This behavior matches that of the mypy command.
Now searches for external files up and down from the current node.  If found, runs pylint on all external Python files found.  If not, runs pylint on the tree headed by the current node.
Copy link
Member

@edreamleo edreamleo left a comment

Choose a reason for hiding this comment

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

@tbpassin Excellent PR. Well done.

@edreamleo edreamleo merged commit a9965df into leo-editor:devel Feb 10, 2023
@edreamleo
Copy link
Member

edreamleo commented Feb 10, 2023

@tbpassin rev 7d0eb2c in devel fixes a mypy complaint and removes trailing whitespace in a few lines. No big deal.

In the future, please run the equivalent of full-test-leo.cmd before committing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants