Skip to content

Conversation

@nirinchev
Copy link
Collaborator

@nirinchev nirinchev commented Jan 7, 2025

Description

This cleans up a few gha workflows:

  1. Format using prettier (best review this with whitespace: off)
  2. Bump checkout and setup-node versions to latest
  3. Configure token/author in the update-electron workflow
  4. Removes a non-existent secret from the start-ga workflow

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

with:
# don't checkout a detatched HEAD
ref: ${{ github.head_ref }}
token: ${{ steps.app-token.outputs.token }}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The changes to this workflow are an attempt to disassociate @himanshusinghs from the generated commit.

- name: Start Release
env:
JIRA_API_TOKEN: ${{ secrets.JIRA_API_TOKEN }}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This secret doesn't seem to exist and it's not used anywhere.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Good change! Seems like we have something in our precommit hook logic though preventing this from being done automatically, probably worth a ticket 🤔

@nirinchev nirinchev added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Jan 7, 2025
@nirinchev nirinchev merged commit 8734e7f into main Jan 7, 2025
27 of 30 checks passed
@nirinchev nirinchev deleted the ni/workflow-updates branch January 7, 2025 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants