Skip to content

Conversation

@TomStrepsil
Copy link
Contributor

@TomStrepsil TomStrepsil commented Feb 27, 2025

Issue

26

Details

Fixing publishing pipline, as per NPM and GitHub documentation, mirroring prior art from @asos/snyker.

For good measure, updated all package.json to have a link back to the repository, again as per @asos/snkyer. Unsure if needed for NPM packages, but it's suggested for GitHub Packages and probably good metadata to have in NPM in any case.

Whilst there, also inspired by @asos/snyker, added a bugs and directories/doc section, again to help link packages back to the code repository.

As per #21 - some unexpected changes to playwright snapshots found, blocking CI. This PR temporarily updates these, whilst waiting on #29

CheckList

  • PR starts with [ISSUE_ID].
  • Has been tested (where required) before merge to main.

@TomStrepsil TomStrepsil changed the title Fix/public publishing [26] Fix public/scoped package publishing Feb 27, 2025
{
"name": "web-toggle-point-serve-example",
"version": "0.2.0",
"version": "0.2.3",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

See here for explanation of 0->3 bump.

@asos-danielc asos-danielc merged commit 296a951 into ASOS:main Mar 6, 2025
10 checks passed
@TomStrepsil TomStrepsil deleted the fix/public-publishing branch March 6, 2025 13:09
TomStrepsil added a commit to TomStrepsil/web-toggle-point that referenced this pull request Apr 10, 2025
* update workflows

* version

* typo

* update chromium linux snaps

* versions for serve update

* package.json repository field

* update root package.lock

* bugs & directories/doc fields

* fix changelog

---------

Co-authored-by: Tom Pereira <[email protected]>
asos-dominicjomaa pushed a commit that referenced this pull request Jul 14, 2025
* [#18] Fix JSDoc module names (#19)
* [26] Fix public/scoped package publishing (#27)
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