Skip to content

Apply pylint CI to all ADF files under "lib". - #492

Merged
nusbaume merged 7 commits into
NCAR:mainfrom
nusbaume:pylint_lib
Oct 8, 2026
Merged

nusbaume merged 7 commits into
NCAR:mainfrom
nusbaume:pylint_lib

Conversation

@nusbaume

Copy link
Copy Markdown
Collaborator

This PR updates the pylint CI to now check all files under the lib directory (excluding those under lib/externals which are not officially part of the ADF).

This is an improvement over the original CI script, which only analyzed a subset of core ADF files.

Fixes #4

@nusbaume nusbaume self-assigned this Sep 24, 2026
@nusbaume nusbaume added code clean-up Made code simpler and/or easier to read. testing system Related to the automated testing system scripts or workflow. labels Sep 24, 2026
nusbaume and others added 6 commits October 1, 2026 14:36
Touch every python file directly under 'lib' so that the updated
pylint GitHub Action (which only lints modified files) runs on each.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@nusbaume
nusbaume merged commit 425e777 into NCAR:main Oct 8, 2026
7 checks passed
@nusbaume
nusbaume deleted the pylint_lib branch October 8, 2026 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code clean-up Made code simpler and/or easier to read. testing system Related to the automated testing system scripts or workflow.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Diagnostics needs unit and linter (pylint) tests

1 participant