Skip to content

Conversation

@webwarrior-ws
Copy link
Contributor

To avoid running this step in situations like opening PR from main repository.

@webwarrior-ws webwarrior-ws force-pushed the only-push-nuget-from-master branch from 3f8346e to 6fa7860 Compare July 7, 2025 09:44
@knocte
Copy link
Collaborator

knocte commented Jul 7, 2025

@webwarrior-ws are you sure this will not break the logic for git tags? AFAIR git tags will not signal "master" as REF.

@webwarrior-ws webwarrior-ws force-pushed the only-push-nuget-from-master branch from 6fa7860 to c6e9b06 Compare July 7, 2025 09:52
@webwarrior-ws
Copy link
Contributor Author

@knocte moved check for "master" branch after tag check

To avoid running this step in situations like opening PR from main
repository.
@webwarrior-ws webwarrior-ws force-pushed the only-push-nuget-from-master branch from c6e9b06 to 2786d0c Compare July 7, 2025 10:10
@knocte
Copy link
Collaborator

knocte commented Jul 7, 2025

The deployReleaseDocs failure is the Fornax issue being fixed in #736, which we will merge next, so I'm merging this now; thanks @webwarrior-ws !

@knocte knocte merged commit 6649d1e into fsprojects:master Jul 7, 2025
4 of 5 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