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

Add a build script and update the readme #4

Merged
merged 2 commits into from
Jul 26, 2023
Merged

Conversation

tgolen
Copy link
Contributor

@tgolen tgolen commented Jul 25, 2023

The missing build script is causing the "publish to NPM" workflow to fail. For now, this adds a placeholder build script.

Manual Tests

  1. Once this PR is merged, go to the "Actions" tab in GH
  2. Verify that the publish workflow completes without error
  3. Search NPMjs.org and verify the package was published

@tgolen tgolen requested a review from a team July 25, 2023 19:19
@tgolen tgolen self-assigned this Jul 25, 2023
@melvin-bot melvin-bot bot requested review from cristipaval and removed request for a team July 25, 2023 19:19
@cristipaval cristipaval merged commit f923cf5 into main Jul 26, 2023
2 checks passed
@hayata-suenaga hayata-suenaga deleted the tgolen-fix-publish branch August 2, 2023 22:32
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