Skip to content

ci: snapshot only on default branch#7355

Merged
eradman merged 2 commits intomasterfrom
lucasfcnunes/snapshot-only-on-default-branch
May 1, 2025
Merged

ci: snapshot only on default branch#7355
eradman merged 2 commits intomasterfrom
lucasfcnunes/snapshot-only-on-default-branch

Conversation

@lucasfcnunes
Copy link
Copy Markdown
Member

What type of PR is this?

  • Refactor
  • Feature
  • Bug Fix
  • New Query Runner (Data Source)
  • New Alert Destination
  • Other

Description

How is this tested?

  • Unit tests (pytest, jest)
  • E2E Tests (Cypress)
  • Manually
  • N/A

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

@lucasfcnunes lucasfcnunes self-assigned this Mar 2, 2025
@lucasfcnunes lucasfcnunes requested a review from eradman March 2, 2025 01:57
@eradman eradman force-pushed the lucasfcnunes/snapshot-only-on-default-branch branch from b544756 to 66447c7 Compare May 1, 2025 12:44
Copy link
Copy Markdown
Collaborator

@eradman eradman left a comment

Choose a reason for hiding this comment

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

The effect of this is to skip this action if it was somehow run on another branch
https://github.com/eradman/redash/actions/runs/14775817788

@eradman eradman enabled auto-merge (squash) May 1, 2025 13:08
@eradman eradman merged commit 8e6c02e into master May 1, 2025
15 checks passed
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.

2 participants