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

Fix/tr 5972/npmRelease skipped publish step #130

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

olga-kulish
Copy link
Contributor

@olga-kulish olga-kulish commented Mar 5, 2024

Related to https://oat-sa.atlassian.net/browse/TR-5972

One more fix...

taoRelease npmRelease lost its publish step, because --no-publish parameter was added in #125, but it was passed to release.js only in npmReleaseMonorepo.

Screenshot_36

Copy link

github-actions bot commented Mar 5, 2024

Coverage Report

Totals Coverage
Statements: 46.4% ( 773 / 1666 )
Methods: 35.06% ( 95 / 271 )
Lines: 45.44% ( 423 / 931 )
Branches: 54.96% ( 255 / 464 )

StandWithUkraine

@olga-kulish olga-kulish changed the title fix: npmRelease skipped publish step Fix/tr 5972/npmRelease skipped publish step Mar 5, 2024
@olga-kulish olga-kulish self-assigned this Mar 6, 2024
@olga-kulish olga-kulish merged commit ca59d46 into develop Mar 6, 2024
2 of 4 checks passed
@olga-kulish olga-kulish deleted the fix/TR-5972/npm-release-publish-step-skipped branch March 6, 2024 08:44
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