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

chore: add github workflow for npm publish #1073

Merged
merged 6 commits into from
Jul 10, 2024

Conversation

mrnaveira
Copy link
Collaborator

@mrnaveira mrnaveira commented Jul 10, 2024

Description

Added a new github CI action to automatically publish to npm registry on every version change::

  • TypeScript bindings
  • Wallet daemon client

Motivation and Context

The typescript-bindings and the wallet-daemon-client npm packages are used by multiple projects. We want to automatically publish their latest versions to the NPM registry when the package versions increases. This way dependent projects can directly include the npm dependency for them.

How Has This Been Tested?

In progress

What process can a PR reviewer use to test or verify this change?

Testing in progress

Breaking Changes

  • None
  • Requires data directory to be deleted
  • Other - Please specify

@mrnaveira mrnaveira requested a review from a team as a code owner July 10, 2024 11:37
@mrnaveira mrnaveira marked this pull request as draft July 10, 2024 11:37
@mrnaveira mrnaveira changed the title github ci that publishes to npm registry chore: add github workflow for npm publish Jul 10, 2024
Copy link

Test Results (CI)

540 tests   540 ✅  1h 47m 4s ⏱️
 64 suites    0 💤
  2 files      0 ❌

Results for commit 6ddc4ce.

@mrnaveira mrnaveira marked this pull request as ready for review July 10, 2024 13:58
@sdbondi sdbondi merged commit b3199a6 into tari-project:development Jul 10, 2024
9 of 10 checks passed
sdbondi added a commit that referenced this pull request Jul 10, 2024
Description
---
Fixes the package location problems of
#1073

Motivation and Context
---
Does not apply 

How Has This Been Tested?
---
Does not apply 

What process can a PR reviewer use to test or verify this change?
---
Does not apply 

Breaking Changes
---

- [x] None
- [ ] Requires data directory to be deleted
- [ ] Other - Please specify

---------

Co-authored-by: Stan Bondi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants