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

don't publish to latest docs unless in the default branch #9673

Merged
merged 3 commits into from
Aug 3, 2023

Conversation

@sdelamo sdelamo added the relates-to: build label for issues related to the build file or CI label Aug 3, 2023
@sdelamo sdelamo changed the title for 3.x releases don't publish to latest don't publish to latest docs unless in the default branch Aug 3, 2023
@sdelamo
Copy link
Contributor Author

sdelamo commented Aug 3, 2023

@alvarosanchez sorry I changed the original contents of this PR to now publish only to latest if the release is running in the default branch. Can you review it again?

I tried this in a personal repo:

Non default branch

https://github.com/sdelamo/github-actions-tests/actions/runs/5752560676

default branch:

https://github.com/sdelamo/github-actions-tests/actions/runs/5752507393

I wonder if this will work with tags.

@sdelamo sdelamo merged commit b643a43 into 3.8.x Aug 3, 2023
4 checks passed
@sdelamo sdelamo deleted the sdelamo-patch-8 branch August 3, 2023 15:51
@sonarcloud
Copy link

sonarcloud bot commented Aug 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

warning The version of Java (11.0.19) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relates-to: build label for issues related to the build file or CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants