Skip to content

MSI: Cleanup and improved error handling#1191

Merged
lrandersson merged 1 commit intoconda:briefcase-integrationfrom
lrandersson:dev-ra-briefcase-improvements
Mar 31, 2026
Merged

MSI: Cleanup and improved error handling#1191
lrandersson merged 1 commit intoconda:briefcase-integrationfrom
lrandersson:dev-ra-briefcase-improvements

Conversation

@lrandersson
Copy link
Copy Markdown
Contributor

Description

This PR contains a small set of generic improvements that I wanted to make to account for all the recent updates. This removes some redundancy and contains improved error handling.

  1. create() function improvements:
  • Changed generic Exception to OSError for platform check (OS-level constraint)
  • Changed generic Exception to ValueError for missing logging support (invalid configuration)
  • Moved briefcase existence check before payload.prepare() to fail fast
  • Added try/finally to ensure payload.remove() cleanup runs even on failure
  1. Payload.prepare() cleanup:
  • Removed unused return value (root, external_dir, base_dir, pkgs_dir) - it didn't make any sense to return paths that don't exist (due to the archiving)

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

@lrandersson lrandersson requested a review from a team as a code owner March 30, 2026 14:51
@github-project-automation github-project-automation bot moved this to 🆕 New in 🔎 Review Mar 30, 2026
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Mar 30, 2026
@lrandersson lrandersson changed the base branch from main to briefcase-integration March 30, 2026 14:52
@lrandersson lrandersson self-assigned this Mar 30, 2026
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Approved in 🔎 Review Mar 30, 2026
@lrandersson lrandersson merged commit 89b0ed5 into conda:briefcase-integration Mar 31, 2026
36 of 37 checks passed
@github-project-automation github-project-automation bot moved this from ✅ Approved to 🏁 Done in 🔎 Review Mar 31, 2026
lrandersson added a commit that referenced this pull request Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed [bot] added once the contributor has signed the CLA

Projects

Status: 🏁 Done

Development

Successfully merging this pull request may close these issues.

3 participants