← Back to issue list

style(lint): bump max function args to 8

View original Github issue

Metadata

Project
starbase
Number
#149
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged into main after approval and passing CI. Updated lint configuration to increase the maximum allowed function arguments to eight, resolving issue #133. Code coverage remained unchanged.

Suggested action: —

No scores available.

Issue body

fixes #133 - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged into main after approval and passing CI. Updated lint configuration to increase the maximum allowed function arguments to eight, resolving issue #133. Code coverage remained unchanged.
qwen3.6-35b-a3b-mtp-q6 — — Merged into main, bumping the maximum function arguments to 8. Resolves issue #133 by updating linting rules, maintaining 100% code coverage.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5 style(lint): fix linting issues starflow merged Merged after approval. Resolved linting issues by modifying one file with three additions and three deletions. No unresolved comments or CI checks were present.
74%
#239 style(lint): autoformat starbase merged Merged after approval. Applied automated linting to a single file, adjusting one line. The maintainer requested the merge post-review, completed without CI checks or unresolved comments.
71%
#150 style(lint): private member access starbase merged Merged into main after approval and passing CI checks. The pull request applies linting style fixes for private member access, modifying one file with two added lines. Code coverage remains unchanged.
70%
#4732 feat(lint): migrate lint command snapcraft merged Migrated lint command logic from core22, adding workarounds for core24 availability and glibc linter warnings. Approved by reviewers and merged after passing CI checks.
70%