Skip to content

Conversation

@vbabich
Copy link
Contributor

@vbabich vbabich commented Dec 12, 2025

Without the packages option, Lerna was unable to find unpublished packages when I ran ./node_modules/.bin/lerna publish from-package locally.

@vbabich vbabich self-assigned this Dec 12, 2025
@vbabich vbabich changed the title Add force-publish parameter for canary releases ci: Add force-publish parameter for canary releases Dec 12, 2025
@vbabich vbabich marked this pull request as ready for review December 12, 2025 21:46
@vbabich vbabich changed the title ci: Add force-publish parameter for canary releases ci: Add force-publish parameter for canary releases, use workspaces Dec 12, 2025
@vbabich vbabich requested a review from mofojed December 12, 2025 21:55
@vbabich vbabich marked this pull request as draft December 12, 2025 22:15
@vbabich vbabich marked this pull request as ready for review December 12, 2025 22:23
@vbabich vbabich changed the title ci: Add force-publish parameter for canary releases, use workspaces ci: Add force-publish parameter for canary releases, add packages path Dec 12, 2025
@mofojed
Copy link
Member

mofojed commented Dec 12, 2025

@vbabich with publish alpha were we able to select a particular package to publish or did it always publish them all?

@vbabich
Copy link
Contributor Author

vbabich commented Dec 12, 2025

@vbabich with publish alpha were we able to select a particular package to publish or did it always publish them all?

Hm... I think we published the packages that had changes, there was no selection.

I got 'force-publish' from deephaven/web-client-ui@7b1c878#diff-f649dfa0883378db13b96d378fbaedfaf99e75a92d0962d813575b79643dd01bR41
but now I think it might not be a good idea to use it here. I'll revert.

@vbabich vbabich changed the title ci: Add force-publish parameter for canary releases, add packages path ci: Add packages path for Lerna Dec 12, 2025
@vbabich vbabich merged commit d75552a into deephaven:main Dec 13, 2025
11 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