dennismdejong opened a new pull request, #13671:
URL: https://github.com/apache/gravitino/pull/13671

   ### What changes were proposed in this pull request?
   
   Configure `max-args` and `max-positional-arguments` in the `[DESIGN]` 
section of the client-python `pylintrc`, consistent with the existing 
`max-returns=10` limit, and remove `too-many-arguments` and 
`too-many-positional-arguments` from the `disable` list.
   
   ### Why are the changes needed?
   
   The client-python DTO constructors, builders and API methods legitimately 
carry up to 10 parameters (mirroring the Java client), so configuring the 
design threshold is the correct resolution instead of degrading the public API 
or leaving the rules disabled.
   
   Fix: #3560
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   ```bash
   ./gradlew clients:client-python:pylint
   ./gradlew clients:client-python:test -PskipITs
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to