Skip to content

Fix for nightly release#853

Merged
danditomaso merged 2 commits into
meshtastic:mainfrom
danditomaso:fix/nightly-gh-action
Sep 14, 2025
Merged

Fix for nightly release#853
danditomaso merged 2 commits into
meshtastic:mainfrom
danditomaso:fix/nightly-gh-action

Conversation

@danditomaso

Copy link
Copy Markdown
Collaborator

Description

This fixes a typo that made its way into the nightly github action workflow. A typo made its way into the tags step in the github action which caused it to fail while running. This PR removes that typo.

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 14, 2025 18:58
@vercel

vercel Bot commented Sep 14, 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.

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 fixes a typo in the nightly GitHub Actions workflow that was causing the workflow to fail during execution. The fix involves correcting the order and format of output variable assignments in the tags computation step.

  • Removes the all_tags output variable that was incorrectly formatted
  • Reorders the remaining output variables for better readability
  • Maintains the same functionality while fixing the workflow failure
Comments suppressed due to low confidence (2)

.github/workflows/nightly.yml:95

  • The all_tags output variable is being referenced but was removed in the tags computation step. This will cause the workflow to fail as the variable no longer exists.
          tags: ${{ steps.meta.outputs.all_tags }}

.github/workflows/nightly.yml:109

  • The all_tags output variable is being referenced but was removed in the tags computation step. This will cause the workflow to fail as the variable no longer exists.
          tags: ${{ steps.meta.outputs.all_tags }}

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

@danditomaso danditomaso merged commit eb2791e into meshtastic:main Sep 14, 2025
1 of 3 checks passed
@danditomaso danditomaso deleted the fix/nightly-gh-action branch September 14, 2025 19: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