Skip to content

Conversation

@Saadnajmi
Copy link
Collaborator

Summary:

prerelease can be undefined while generatorOptions.preid is an empty string. Update the check to handle that

Test Plan:

CI should pass

@Saadnajmi Saadnajmi requested a review from a team as a code owner March 24, 2025 00:45
@Saadnajmi Saadnajmi changed the title ci: update prepublish check to check prerelease existence ci: update preid check in prepublish.mjs Mar 24, 2025
@Saadnajmi Saadnajmi merged commit 766ff79 into microsoft:main Mar 25, 2025
12 checks passed
Saadnajmi added a commit to Saadnajmi/react-native-macos that referenced this pull request Mar 25, 2025
`prerelease` can be `undefined` while `generatorOptions.preid` is an
empty string. Update the check to handle that

CI should pass
Saadnajmi added a commit that referenced this pull request Mar 25, 2025
## Summary:

Backport of #2432 to 0.78-stable

## Test Plan:

CI should pass
Saadnajmi added a commit to Saadnajmi/react-native-macos that referenced this pull request Mar 26, 2025
## Summary:

`prerelease` can be `undefined` while `generatorOptions.preid` is an
empty string. Update the check to handle that

## Test Plan:

CI should pass
Saadnajmi added a commit to Saadnajmi/react-native-macos that referenced this pull request Mar 26, 2025
## Summary:

`prerelease` can be `undefined` while `generatorOptions.preid` is an
empty string. Update the check to handle that

## Test Plan:

CI should pass
Saadnajmi added a commit that referenced this pull request Mar 26, 2025
## Summary:

Backport #2422 , #2432 , and #2434 to 0.76-stable

## Test Plan:

CI should pass

---------

Co-authored-by: Tommy Nguyen <4123478+tido64@users.noreply.github.com>
Saadnajmi added a commit that referenced this pull request Mar 26, 2025
## Summary:

Backport #2422 , #2432 , and #2434 to 0.77-stable

## Test Plan:

CI should pass

---------

Co-authored-by: Tommy Nguyen <4123478+tido64@users.noreply.github.com>
@Saadnajmi Saadnajmi mentioned this pull request May 1, 2025
4 tasks
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