Skip to content

[26] Fix public/scoped package publishing #27

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

Merged
merged 9 commits into from
Mar 6, 2025

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
@@ -1,6 +1,6 @@
{
"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