Skip to content

fix: deploy trigger#48

Merged
bernacodesido merged 2 commits intorootstockfrom
fix/deployTrigger
Jun 3, 2025
Merged

fix: deploy trigger#48
bernacodesido merged 2 commits intorootstockfrom
fix/deployTrigger

Conversation

@bernacodesido
Copy link
Copy Markdown
Member

Deploy was triggered twice. This PR fixes that.

Copilot AI review requested due to automatic review settings May 30, 2025 20:59
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 30, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Copy link
Copy Markdown

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 addresses the issue of duplicate deploy triggers by modifying the release event types in the deploy workflow.

  • Removed the 'created' event from the release trigger to prevent duplicate deployments.

on:
release:
types: [created, edited, published]
types: [edited, published]
Copy link

Copilot AI May 30, 2025

Choose a reason for hiding this comment

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

Consider adding an inline comment explaining that the 'created' event was removed to avoid duplicate deploy triggers, to improve maintainability and clarity.

Copilot uses AI. Check for mistakes.
@bernacodesido bernacodesido merged commit 1f598ad into rootstock Jun 3, 2025
6 checks passed
@bernacodesido bernacodesido deleted the fix/deployTrigger branch June 9, 2025 17:52
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