Skip to content

Conversation

@michalsek
Copy link
Member

@michalsek michalsek commented Sep 19, 2025

Simplified github actions workflow files for easier usage with NPM OIDC configuration

Closes RNAA-253

⚠️ Breaking changes ⚠️

Introduced changes

  • removes nigtly build workflow in favor of using just npm-audio-package-build.yml

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added/Conducted relevant tests
  • Performed self-review of the code
  • Updated Web Audio API coverage
  • Added support for web

Copy link
Contributor

Copilot AI left a comment

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 simplifies the GitHub Actions workflow configuration by consolidating the nightly build functionality into a single workflow file. The main purpose is to streamline NPM OIDC configuration and reduce workflow complexity.

  • Removes the separate nightly build workflow file
  • Consolidates scheduling and manual dispatch triggers into the main build workflow
  • Updates input parameter naming for better clarity

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/npm-publish-nightly.yml Completely removed - eliminates separate nightly workflow
.github/workflows/npm-audio-package-build.yml Enhanced with scheduling and updated parameter logic for consolidated functionality

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

@michalsek michalsek merged commit 78fe1dc into main Sep 19, 2025
3 checks passed
@michalsek michalsek deleted the chore/simplified-publish-ci branch September 19, 2025 12:24
michalsek added a commit that referenced this pull request Oct 6, 2025
* chore: simplify audio-api publish scripts to single file

* chore: update default runner time

* chore: use environments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants