Skip to content

add rubyzip gem#897

Merged
zetter-rpf merged 1 commit into
mainfrom
add-zip-gem
Jun 26, 2026
Merged

add rubyzip gem#897
zetter-rpf merged 1 commit into
mainfrom
add-zip-gem

Conversation

@rammodhvadia

@rammodhvadia rammodhvadia commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

adds rubyzip gem, used in sb3_parser.rb

@cla-bot cla-bot Bot added the cla-signed label Jun 26, 2026
@github-actions

Copy link
Copy Markdown

Test coverage

91.97% line coverage reported by SimpleCov.
Run: https://github.com/RaspberryPiFoundation/editor-api/actions/runs/28235089320

@rammodhvadia rammodhvadia marked this pull request as ready for review June 26, 2026 11:30
Copilot AI review requested due to automatic review settings June 26, 2026 11:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds rubyzip as an explicit dependency so the runtime SB3 import/parsing code (Sb3Parser/ProjectImporter) can reliably use require 'zip' regardless of transitive dependencies.

Changes:

  • Add gem 'rubyzip' to Gemfile.
  • Update Gemfile.lock to include rubyzip in top-level dependencies.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
Gemfile Adds rubyzip as a direct dependency for SB3 parsing support.
Gemfile.lock Records rubyzip as a direct dependency (lock already contains the resolved gem/spec entry).

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

@zetter-rpf zetter-rpf merged commit 8ee29e2 into main Jun 26, 2026
7 checks passed
@zetter-rpf zetter-rpf deleted the add-zip-gem branch June 26, 2026 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants