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.

Sign up for GitHub

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

Jump to bottom

chore: separate build target for release build #1901

Merged
merged 5 commits into from
Aug 27, 2024
Merged

chore: separate build target for release build #1901

merged 5 commits into from
Aug 27, 2024

Conversation

aixaCode
Copy link
Collaborator

What does this PR change?

  • For release build we would use new build target to avoid blocking the main
  • Don't run test if not modifying anything from Explorer folder

Our Code Review Standards

https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md

@aixaCode aixaCode added the no QA needed Used to tag pull requests that does not require QA validation label Aug 27, 2024
@github-actions GitHub Actions
Copy link

badge

New build in progress, come back later!

1 similar comment
@github-actions GitHub Actions
Copy link

badge

New build in progress, come back later!

@aixaCode aixaCode enabled auto-merge (squash) August 27, 2024 14:11
@aixaCode aixaCode disabled auto-merge August 27, 2024 14:20
@aixaCode aixaCode enabled auto-merge (squash) August 27, 2024 15:00
@aixaCode aixaCode merged commit 2513be8 into main Aug 27, 2024
5 checks passed
@aixaCode aixaCode deleted the chore/unity-cloud-release-build-new-target branch August 27, 2024 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m3taphysics m3taphysics m3taphysics approved these changes

@davidejensen davidejensen davidejensen approved these changes

@fcolarich fcolarich fcolarich approved these changes

Assignees
No one assigned
Labels
no QA needed Used to tag pull requests that does not require QA validation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aixaCode @m3taphysics @davidejensen @fcolarich