Skip to content

Improve the nightly and release workflow#836

Merged
danditomaso merged 4 commits into
meshtastic:mainfrom
danditomaso:fix/publishing-fixes
Sep 11, 2025
Merged

Improve the nightly and release workflow#836
danditomaso merged 4 commits into
meshtastic:mainfrom
danditomaso:fix/publishing-fixes

Conversation

@danditomaso

Copy link
Copy Markdown
Collaborator

Description

This PR contains several fixes for long standing issues. The first is a focus on improving how our release workflow operates, its been failing at different spots and this (hopefully) should fix the remaining issues with releasing packages and our web client. The next is a fix for package publishing, ensuring that we expose the correct files within the package.json and jsr.json file. Lastly, our nightly workflow has been experiencing problems for a while and this should clean up any existing issues with that process.

Checklist

  • Code follows project style guidelines
  • Documentation has been updated or added
  • Tests have been added or updated
  • All i18n translation labels have been added (read
    CONTRIBUTING_I18N_DEVELOPER_GUIDE.md for more details)

Copilot AI review requested due to automatic review settings September 7, 2025 20:50
@vercel

vercel Bot commented Sep 7, 2025

Copy link
Copy Markdown

@danditomaso is attempting to deploy a commit to the Meshtastic Team on Vercel.

A member of the Team first needs to authorize it.

@danditomaso danditomaso requested a review from philon- September 7, 2025 20:51

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

This PR improves the CI/CD workflow infrastructure by fixing issues with package publishing and container image building. The changes focus on standardizing file extensions, correcting container image configurations, and streamlining release workflows.

  • Standardizes package.json entry points from .mjs to .js extensions across all packages
  • Updates container image build processes and nginx base image version
  • Simplifies and fixes workflow configurations for release and nightly builds

Reviewed Changes

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

Show a summary per file
File Description
packages/web/infra/Containerfile Updates nginx base image and fixes file paths for container build
packages/*/package.json Standardizes entry point file extensions from .mjs/.mts to .js/.ts
.github/workflows/release-web.yml Simplifies workflow logic and switches to docker/build-push-action
.github/workflows/release-packages.yml Adds version bumping, improves error handling, and streamlines publishing
.github/workflows/nightly.yml Fixes tag reference and removes working directory specification

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/workflows/nightly.yml Outdated
Comment thread .github/workflows/nightly.yml Outdated
@vercel

vercel Bot commented Sep 8, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
web-test Ready Ready Preview Comment Sep 9, 2025 8:35pm

Comment thread .github/workflows/nightly.yml Outdated
Comment thread .github/workflows/nightly.yml
danditomaso and others added 2 commits September 9, 2025 15:14
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Comment thread packages/transport-node-serial/package.json
@danditomaso danditomaso requested a review from bergie September 11, 2025 19:24

@bergie bergie 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.

The package.json changes look good. The commit date stuff we've discussed separately on Discord.

@danditomaso

Copy link
Copy Markdown
Collaborator Author

The package.json changes look good. The commit date stuff we've discussed separately on Discord.

Thanks!

@danditomaso danditomaso merged commit 765f672 into meshtastic:main Sep 11, 2025
2 of 3 checks passed
@danditomaso danditomaso deleted the fix/publishing-fixes branch September 11, 2025 19:34
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.

4 participants