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

test snapit #498

Closed
wants to merge 1 commit into from
Closed

test snapit #498

wants to merge 1 commit into from

Conversation

robin-drexler
Copy link
Member

No description provided.

@robin-drexler
Copy link
Member Author

/snapit

1 similar comment
@robin-drexler
Copy link
Member Author

/snapit

@simontaisne
Copy link
Member

/snapit

@simontaisne
Copy link
Member

@robin-drexler snapit is running into E404 Not Found errors when trying to publish the remote-dom/core, @remote-dom/preact, and @remote-dom/signals packages to NPM. Do you think it might be because we're missing the NPM_TOKEN environment variable?

GITHUB_TOKEN: ${{ secrets.SHOPIFY_GH_ACCESS_TOKEN }}
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}

@robin-drexler
Copy link
Member Author

@robin-drexler snapit is running into E404 Not Found errors when trying to publish the remote-dom/core, @remote-dom/preact, and @remote-dom/signals packages to NPM. Do you think it might be because we're missing the NPM_TOKEN environment variable?

GITHUB_TOKEN: ${{ secrets.SHOPIFY_GH_ACCESS_TOKEN }}
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}

yeah it doesn't work (yet)

We had to merge the action to actually try it.

I'm not sure yet why it's not working, but noticed the 404.

We do feed it a token: https://github.com/Shopify/remote-dom/blob/main/.github/workflows/snapit.yml#L23, but maybe it's not the correct one.
It's the same one we use in other places, but as NODE_AUTH_TOKEN.

github actions are unfortunately quite difficult to test.

@robin-drexler
Copy link
Member Author

/snapit

1 similar comment
@robin-drexler
Copy link
Member Author

/snapit

@shopify-github-actions-access
Copy link
Contributor

🫰✨ Thanks @robin-drexler! Your snapshots have been published to npm.

Test the snapshots by updating your package.json with the newly published versions:

"@remote-dom/core": "0.0.0-snapshot-20241120193113",
"@remote-dom/preact": "0.0.0-snapshot-20241120193113",
"@remote-dom/signals": "0.0.0-snapshot-20241120193113"

@robin-drexler
Copy link
Member Author

/snapit

@shopify-github-actions-access
Copy link
Contributor

🫰✨ Thanks @robin-drexler! Your snapshots have been published to npm.

Test the snapshots by updating your package.json with the newly published versions:

"@remote-dom/core": "0.0.0-snapshot-20241120201825",
"@remote-dom/polyfill": "0.0.0-snapshot-20241120201825",
"@remote-dom/preact": "0.0.0-snapshot-20241120201825",
"@remote-dom/react": "0.0.0-snapshot-20241120201825",
"@remote-dom/signals": "0.0.0-snapshot-20241120201825"

@simontaisne
Copy link
Member

Nice!

@lemonmade lemonmade closed this Nov 21, 2024
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.

3 participants