← Back to issue list

fix(charm-plugin): remove existence checks for reqs

View original Github issue

Metadata

Project
charmcraft
Number
#1808
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged canonical/charmcraft#1808, removing charm-plugin requirements-file existence checks so initialization works before those files are created (commit 0d3b2713).

Resolution: closed resolved

Reason: PR #1808 was merged as commit 0d3b2713. It removed checks for requirements files during charm-plugin instantiation because those files may not exist yet.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Requirements files might not exist yet when the charm plugin is instantiated, so don't check for their existence in the plugin.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged canonical/charmcraft#1808, removing charm-plugin requirements-file existence checks so initialization works before those files are created (commit 0d3b2713).
stealth/space-bunny-alpha — closed resolved Merged canonical/charmcraft#1808, removing charm-plugin requirements-file existence checks so initialization works before those files are created.
stealth/space-bunny-alpha — closed resolved Merged PR removed requirement-file existence checks from the charm plugin, allowing plugin creation before those files exist.
qwen/qwen3.6-35b-a3b — — Merged. Removed existence checks for requirements files in the charm plugin to prevent errors when files are not yet instantiated.
qwen3.6-35b-a3b-mtp-q6 — — Merged a fix to the charm plugin that removes existence checks for requirements files, as they may not be present during initialization.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#468 parts: add entrypoint and requirements properties to charm plugin (CRAFT-393) charmcraft merged Merged as d892b3f3, adding entrypoint and requirements plugin properties, deprecating equivalent CLI flags, and rejecting dual configuration. A regression was later fixed in #472.
71%