fix(provider): don't pass null values or certain excluded values from config
Metadata
Current evaluation
Merged fix preventing null and excluded configuration values from being passed to the provider. Approved by two reviewers, passed all CI checks, and updated the changelog.
Suggested action: —
No scores available.
Issue body
- [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 `make lint && make test`?
- [x] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged fix preventing null and excluded configuration values from being passed to the provider. Approved by two reviewers, passed all CI checks, and updated the changelog. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix preventing null and excluded configuration values from being passed to the provider. The maintainer confirmed linting, tests, and changelog updates prior to merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #364 fix: ensure provider is available before launching | craft-application | merged | Merged a fix by maintainer cmatsuoka ensuring the provider is available before launch. Approved by three reviewers and passed CI. The two-line change in one file resolves potential launch failures caused by missing providers. | |
| #1763 fix(provider): forward shared cache | charmcraft | merged | Merged provider fix to forward the shared cache, resolving issue #1762. Approved by two reviewers, the change adds 107 lines across two files to implement the forwarding logic. | |
| #865 fix(provider): bump compatibility tag | craft-application | merged | Merged a fix bumping the Provider service compatibility tag in craft-application to prevent reuse of instances created before the State service. Approved by three reviewers and passed all CI checks. | |
| #130 fix(test): fix provider integration tests | craft-application | merged | Merged to main. Fixed provider integration tests by modifying one file (+3/-9 lines). Achieved 100% diff coverage with no impact on overall metrics. Approved by two reviewers and passed CI checks prior to merge. |