Skip to content

add missing types build to publishing action#3074

Open
danoswaltCL wants to merge 3 commits intorelease/6.4from
bugfix/add-missing-types-build-to-publish-action
Open

add missing types build to publishing action#3074
danoswaltCL wants to merge 3 commits intorelease/6.4from
bugfix/add-missing-types-build-to-publish-action

Conversation

@danoswaltCL
Copy link
Copy Markdown
Collaborator

No description provided.

@danoswaltCL danoswaltCL requested a review from bcb37 April 15, 2026 20:06
Copy link
Copy Markdown
Collaborator

@bcb37 bcb37 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't going to work now that there's no package-lock.json in the directory. We have to install the deps from the top-level using yarn.

@danoswaltCL danoswaltCL force-pushed the bugfix/add-missing-types-build-to-publish-action branch from 297f336 to cd0e217 Compare April 16, 2026 15:04
@danoswaltCL
Copy link
Copy Markdown
Collaborator Author

%&$()&$) why the f are these tests failing now, the changes are all in the .github and clientlib folders. i just want to publish this stupid package

@danoswaltCL
Copy link
Copy Markdown
Collaborator Author

ok there we go. i tested out these steps locally to emulate and it seems like it should work:

yarn install --frozen-lockfile
cd /clientlibs/js
yarn install --frozen-lockfile in clientlibs/js
yarn build
npm publish --dry-run

@bcb37

@danoswaltCL danoswaltCL requested a review from bcb37 April 16, 2026 16:31
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