Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: dont run preview on main #900

Merged
merged 1 commit into from
Dec 6, 2022
Merged

ci: dont run preview on main #900

merged 1 commit into from
Dec 6, 2022

Conversation

vacas5
Copy link
Contributor

@vacas5 vacas5 commented Dec 6, 2022

Description

I keep getting emails about this workflow failing on main. We don't need it to run on main. I am not sure why it is running because that workflow that prompts it shouldn't run on main, but hoping this fixes it.

Which issue(s) does this PR relate to?

Testing

Trade-offs

Screenshots

Checklist

  • This PR is associated with a JIRA and it is mentioned in the commit message footer ("Closes …")
  • Significant changes have been tested downstream to avoid breaking changes
  • This PR contains breaking changes and states in the commit message body ("BREAKING CHANGE: …")
  • I have reviewed the changes and provided detail to the sections above

Copy link
Collaborator

@nataliepina nataliepina left a comment

Choose a reason for hiding this comment

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

Changes seem reasonable. Let's try it out.

@vacas5 vacas5 merged commit a90ff75 into main Dec 6, 2022
@vacas5 vacas5 deleted the randerson/ci/uffizi branch December 6, 2022 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants