Skip to content

Add notify workflow - #165

Merged
codebycarson merged 1 commit into
mainfrom
feature/notify-viem
May 7, 2024
Merged

Add notify workflow#165
codebycarson merged 1 commit into
mainfrom
feature/notify-viem

Conversation

@codebycarson

Copy link
Copy Markdown
Collaborator

This adds a github workflow dispatch event to tell the registry API to update.

This adds a github workflow dispatch event to tell the registry API to update.
@codebycarson
codebycarson requested a review from besatedMay 1, 2024 18:05
@changeset-bot

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: d0e8720

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.56%. Comparing base (ecc4007) to head (d0e8720).
Report is 3 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #165 +/- ##
=======================================
Coverage 84.56% 84.56% =======================================
Files 17 17 Lines 162 162 Branches 25 25 =======================================
Hits 137 137 Misses 25 25 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

- name: Delay for 90 seconds to allow NPM package to propagate
run: sleep 90

- name: Update registry API

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

so this dispatches an event to the registry api to update it?

Sign up for freeto 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.

3 participants

@codebycarson@codecov-commenter@besated