Skip to content

Package updates#761

Merged
lacraig2 merged 4 commits intomainfrom
package_updates
Apr 1, 2026
Merged

Package updates#761
lacraig2 merged 4 commits intomainfrom
package_updates

Conversation

@zestrada
Copy link
Copy Markdown
Member

This PR makes a few small tweaks to the packaging system introduced in #760:

  • adds an unpackage/import command
  • adds the project basename to metadata for unpackaging
    • which also checks to ensure we have a metadata file before unpackaging

I had thought about making the package command create a subdir for a more conventional archive format, but we settled on adding the project base_name to the metadata to have a package that's easier to inspect.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the packaging workflow by extending package metadata and adding an unpacking command to restore packaged projects into a workspace directory.

Changes:

  • Bump package metadata format_version and add base_name to .penguin_packaged_version.
  • Add penguin unpackage to extract a packaged project into an output directory (with --force overwrite support).
  • Add hidden penguin import alias for unpackage.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lintly has detected code quality issues in this pull request.

@zestrada zestrada dismissed github-actions[bot]’s stale review April 1, 2026 13:04

flake8 passes, not sure why this is sticking around. Apologies if I'm missing something

@lacraig2 lacraig2 merged commit 162e81a into main Apr 1, 2026
14 checks passed
@zestrada zestrada deleted the package_updates branch April 1, 2026 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants