Skip to content

Update to remarkable-update-image 1.3.3#19

Merged
Eeems merged 19 commits into
mainfrom
update
Apr 3, 2026
Merged

Update to remarkable-update-image 1.3.3#19
Eeems merged 19 commits into
mainfrom
update

Conversation

@Eeems
Copy link
Copy Markdown
Contributor

@Eeems Eeems commented Apr 3, 2026

Summary by CodeRabbit

  • Chores

    • Released version 1.3.3 and added MIT license metadata.
    • Updated dependency pin: remarkable-update-image → 1.3.3.
  • CI

    • Added a lint job, modernized workflow actions and caching, and disabled matrix fail-fast.
    • Added a dedicated universal-wheel build job, consolidated artifact merging/downloads, and removed one executable build from the workflow.
    • Standardized executable artifact naming and uploads.
  • Build

    • Produces both a universal wheel and a native/platform wheel; install and release flows updated.
    • Venv setup installs build tooling and lint-fix now auto-fixes.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 3, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 15fd31b9-81d2-42cf-8e8e-4f3350893328

📥 Commits

Reviewing files that changed from the base of the PR and between 52a06d5 and 25c065b.

📒 Files selected for processing (1)
  • .github/workflows/build.yml

📝 Walkthrough

Walkthrough

Added a lint CI job, refactored CI matrices and artifact handling, split wheel builds into universal (wheel) and platform (native-wheel), updated Makefile targets/venv bootstrap and lint commands, bumped project version and license metadata, and updated a pinned dependency.

Changes

Cohort / File(s) Summary
CI / Workflows
/.github/workflows/build.yml
Added lint job; added build-any-wheel for universal wheels; refactored Python version matrices using a shared anchor and fail-fast: false; upgraded actions and apt-cache action; adjusted manylinux/docker build to produce native-wheel; renamed per-OS executables before upload; disabled build-executable-remarkable; consolidated artifact upload/download using pattern: pip-* / rmufuse-* with merge-multiple: true; simplified publish and release.
Build System / Makefile
Makefile
Expanded .PHONY into individual target declarations; introduced native-wheel (platform/ABI wheel) and redefined wheel as universal (py3-none-any); release and install now depend on native-wheel; build/sdist/wheel rules run python -m build inside venv with PIP_EXTRA_INDEX_URL set; venv bootstrap installs build; lint-fix uses ruff --fix; added LICENSE to distribution OBJ deps.
Project Metadata
pyproject.toml
Bumped version 1.3.2 → 1.3.3; added license = "MIT" and license-files = ["LICENSE"]; removed MIT license classifier; raised minimum setuptools to >=77.1.
Dependencies
requirements.txt
Updated pinned dependency remarkable-update-image 1.3.2 → 1.3.3.

Sequence Diagram(s)

sequenceDiagram
    participant Dev as Developer
    participant GH as GitHub Actions
    participant Lint as Lint Job
    participant Test as Test Matrix
    participant Native as Native Wheel Builder
    participant AnyWheel as Universal Wheel Builder
    participant Art as Artifact Storage
    participant Publish as Publish Job
    participant Release as Release Job

    Dev->>GH: push / open PR
    GH->>Lint: run lint/format job
    GH->>Test: run test matrix jobs
    GH->>Native: run native-wheel builds (manylinux/ubuntu/alpine)
    GH->>AnyWheel: run build-any-wheel (universal wheel)
    Native->>Art: upload `rmufuse-*` artifacts
    AnyWheel->>Art: upload `pip-*` artifacts
    Art->>Publish: merged download (pattern: pip-*)
    Publish->>Publish: publish pip artifacts
    Art->>Release: merged downloads (patterns: pip-*, rmufuse-*)
    Release->>Release: upload assets to GitHub Release
Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the primary change: updating the remarkable-update-image dependency to version 1.3.3, which is the core objective evident across all modified files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update

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.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

@Eeems Eeems merged commit e1ede5a into main Apr 3, 2026
6 of 7 checks passed
@Eeems Eeems deleted the update branch April 3, 2026 07:33
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