Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

Shopify/cli

shauns created a review on a pull request on Shopify/cli

View on GitHub

github-actions[bot] created a comment on a pull request on Shopify/cli
<!-- jest coverage report action for options with hash 4c542401e6e51275fe98fdb0130d9f77 --> ## Coverage report | <div title="Status of coverage:&#10; 🟢 - ok&#10; 🟡 - slightly more than thresho...

View on GitHub

isaacroldan deleted a branch Shopify/cli

app-watcher-as-process

github-actions[bot] created a comment on a pull request on Shopify/cli
We detected some changes at packages/*/src and there are no updates in the .changeset. If the changes are user-facing, run "pnpm changeset add" to track your changes and include them in the next re...

View on GitHub

shauns created a review on a pull request on Shopify/cli

View on GitHub

isaacroldan pushed 25 commits to improve-extension-validations-errors-in-dev Shopify/cli
  • shallow copy top level fields so we don't change nested keys a0f424a
  • Remove PHP template from CLI Update review comments typo 652e063
  • Changeset 93d65a6
  • Fix password validation to handle capitalized store names on `shopify theme dev` 9548a34
  • Minor wording improvements * "be an option" becomes "be optional" because it will continue to be an option to incl... c914464
  • Fix typo d90036a
  • Minor grammatical nitpick Co-authored-by: jenstanicak <[email protected]> 0a88826
  • Merge pull request #4849 from Shopify/fix-warning-wording Minor wording improvements 312ceca
  • Merge pull request #4798 from Shopify/ms.fix-camel-case-on-input-output [ShopifyVM] camel case top fields only for w... 300c6aa
  • update readme a3d33c0
  • update docs 8554d02
  • Update `shopify theme dev` to no longer display redundant URLs 1e267a2
  • Merge pull request #4847 from Shopify/store-password-validation Fix password validation to handle capitalized store ... c393d92
  • Revert "Revert "Improve binary run safety"" 64dab32
  • Drop support for local CLI switchover (#4842) ### WHY are these changes introduced? Drops support for the local-mod... 6296c65
  • Merge pull request #4854 from Shopify/revert-4527-revert-4401-improve-binary-run-safety Improve binary run safety II 2173363
  • Add import admin link extensions to CLI and migration on deployment 8eb100a
  • Merge pull request #4738 from Shopify/1680-cli-import-link-extension Add import admin link extensions to CLI and mig... 4f35f37
  • Update generated docs 169c93b
  • Merge pull request #4843 from Shopify/liz/remove-php-template Remove PHP template from CLI 79f78cc
  • and 5 more ...

View on GitHub

isaacroldan created a review on a pull request on Shopify/cli

View on GitHub

isaacroldan pushed 1 commit to dont-load-the-app-on-init Shopify/cli

View on GitHub

isaacroldan pushed 1 commit to dont-load-the-app-on-init Shopify/cli

View on GitHub

graphite-app[bot] created a review comment on a pull request on Shopify/cli
`forEach` with async operations won't wait for all promises to resolve. Use `Promise.all(errors.map())` to ensure all errors are processed before the function returns. This avoids potential race co...

View on GitHub

graphite-app[bot] created a review on a pull request on Shopify/cli

View on GitHub

graphite-app[bot] created a review on a pull request on Shopify/cli

View on GitHub

isaacroldan pushed 1 commit to improve-extension-validations-errors-in-dev Shopify/cli
  • Correctly process dev-session errors b6d9afb

View on GitHub

karreiro deleted a branch Shopify/cli

ui-feedback

karreiro pushed 1 commit to ui-feedback Shopify/cli
  • Update 'Share your theme preview' link 2397cd5

View on GitHub

github-actions[bot] closed a draft pull request on Shopify/cli
[DNM] PR to look at -- Rough prototype for extension sourcemaps
Opening PR for a place to put comments.
jamesmengo created a comment on a pull request on Shopify/cli
@jeffreyguenther The JSDoc is not getting re-exported from the CLI package atm, so if you need a reference of what the functions accept, you can refer here - [push.ts](https://github.com/Sho...

View on GitHub

jamesmengo created a review on a pull request on Shopify/cli

View on GitHub

jamesmengo created a review on a pull request on Shopify/cli

View on GitHub

jamesmengo created a comment on an issue on Shopify/cli
Hey @meincocktailfass-tech! I think the `/admin` path is messing up the authentication flow. Could you try `shopify theme dev --store cocktailfassstaging.myshopify.com` instead?

View on GitHub

amcaplan opened a pull request on Shopify/cli
Display link as plaintext if link label is nonexistent or just the link itself
<!-- ☝️How to write a good PR title: - Prefix it with [Feature] (if applicable) - Start with a verb, for example: Add, Delete, Improve, Fix… - Give as much context as necessary and as l...
amcaplan pushed 1 commit to render-plaintext-links-when-footnote-looks-silly Shopify/cli

View on GitHub

amcaplan created a branch on Shopify/cli

render-plaintext-links-when-footnote-looks-silly - Build apps, themes, and hydrogen storefronts for Shopify

lucyxiang created a comment on an issue on Shopify/cli
Thank you for raising this @aleksandar89d. I was able to replicate this issue, I'll put this issue in our backlog.

View on GitHub

amcaplan created a review on a pull request on Shopify/cli
I think this change makes sense, however maybe it should encourage us to handle the footnote URLs differently when the URL and text are the same. The footnote system makes sense when there is meant...

View on GitHub

cdarne pushed 77 commits to dev-dash-webhooks-api Shopify/cli
  • Change colors 7ea76a8
  • Add new option to link for new apps 7a5992d
  • Improve logs 92b778e
  • Merge branch 'main' into fix-complex-template-init 2ff7aa8
  • add test to link when isNewApp is true 38c9cf9
  • Merge branch 'main' into consistent-dev-ux-improvements 5ccb1ac
  • Use alternative palette for consistent-dev da7a6e8
  • Remove commented code 32e44d7
  • Refresh session when 401 errors happens 884ab85
  • Add unit tests for the session 604ecc2
  • Add changeset 5aecb3e
  • Update .changeset/dull-shoes-clean.md Co-authored-by: James Meng <[email protected]> 45e9fa7
  • Merge pull request #4790 from Shopify/401-errors Refresh session when 401 errors happens 0572f93
  • Merge pull request #4808 from Shopify/fix-complex-template-init Add new option to link for new apps c5e4ec5
  • Merge branch 'main' into consistent-dev-ux-improvements ae3ca50
  • Notification system 33477dd
  • Fix a test 149a546
  • Print locale changes to debug dd35e0c
  • Fetch notifications from the Shopify CDN 7d1d18a
  • Use helper function 1005d0d
  • and 57 more ...

View on GitHub

elanalynn pushed 2 commits to condition-with-manifest Shopify/cli
  • Skip adding main to assets 9a283fc
  • change sourcepath to module to be consistent with top level naming dbaefc9

View on GitHub

github-actions[bot] created a comment on a pull request on Shopify/cli
🫰✨ **Thanks @jamesmengo! Your snapshot has been published to npm.** Test the snapshot by intalling your package globally: ```bash pnpm i -g @shopify/[email protected] ``` > After ...

View on GitHub

jamesmengo created a comment on a pull request on Shopify/cli
/snapit

View on GitHub

amcaplan created a comment on a pull request on Shopify/cli
I think this is ready to go. Currently it points to `appsConnection`. We can choose to leave it as is, or update `apps` to be a connection as well and then port this over.

View on GitHub

Load more