Skip to content

Conversation

Jason2866
Copy link
Owner

@Jason2866 Jason2866 commented Sep 10, 2025

Summary by CodeRabbit

  • Chores

    • Switched release artifacts to .tar.xz for improved compression; downloads are now provided as framework-arduinoespressif32-*.tar.xz.
    • Updated build and CI pipelines to use xz utilities; removed 7zip/p7zip dependencies on macOS and Linux.
    • Packaging now excludes dotfiles consistently during archive creation.
  • User Impact

    • Use tar/xz tools to extract archives (e.g., tar -Jxf .tar.xz).

Copy link

coderabbitai bot commented Sep 10, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Replaces ZIP/tar.gz packaging with tar.xz across CI workflow and tools. Updates GitHub release asset pattern, build archive path, and CI dependencies to use tar/xz. No other logic or public interfaces changed.

Changes

Cohort / File(s) Change Summary
Build workflow packaging
.github/workflows/parallel_build.yaml
Switched release artifact from framework-arduinoespressif32-*.zip to *.tar.xz; replaced 7z zip command with tar --exclude=.* -Jcf ... for archiving.
Packaging script update
tools/archive-build.sh
Changed pio_archive_path extension from .tar.gz to .tar.xz; replaced 7z-based zip step with tar --exclude=.* -Jcf to produce xz-compressed tarball.
CI dependencies
tools/prepare-ci.sh
macOS: brew install 7zipbrew install xz; Linux: apt install ... p7zip-fullapt install ... xz-utils.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor Dev as Developer
    participant GH as GitHub Actions
    participant Build as Build Steps
    participant Tar as tar/xz
    participant Rel as Release

    Dev->>GH: Push / PR triggers workflow
    GH->>Build: Compile framework
    Build->>Tar: tar --exclude=.* -Jcf framework-arduinoespressif32-<idf>.tar.xz
    Tar-->>Build: .tar.xz artifact
    Build->>Rel: Upload asset matching framework-arduinoespressif32-*.tar.xz
    Rel-->>Dev: Release contains .tar.xz artifact
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

I bundled bits in crunchy xz,
Hopped past zips with breezy ease—
Tarballs twirl like carrot trees,
CI hums with quiet breeze.
Thump-thump—release, please! 🥕✨

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.


📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e1acb00 and 5cd02a4.

📒 Files selected for processing (3)
  • .github/workflows/parallel_build.yaml (2 hunks)
  • tools/archive-build.sh (2 hunks)
  • tools/prepare-ci.sh (1 hunks)
✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 55_xz

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Jason2866 Jason2866 merged commit 778bda0 into v5.5_Arduino Sep 10, 2025
11 of 12 checks passed
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.

1 participant