Skip to content

Conversation

eyevana
Copy link
Collaborator

@eyevana eyevana commented Sep 23, 2025

https://issues.redhat.com/browse/NGUI-321

Summary of changes:

  • Created new NPM organization @rhngui
  • Created an access token and stored it as the NPM_TOKEN secret in this repository
  • Created a new workflow to publish the package to NPM whenever a tag with pattern of v* pushed to repository
  • Externalize React, React-Dom and marked them as peer dependencies

@eyevana eyevana changed the title refactor: remove references to old directory structure feat(NGUI-321): Publish package to NPM Sep 23, 2025
@eyevana
Copy link
Collaborator Author

eyevana commented Sep 23, 2025

@Hrithik-Gavankar @anujrajak what are your thoughts on externalizing Patternfly dependencies and its CSS. I think this package should be super lightweight and right now I'm getting a little over 2MB unzipped with npx vite-bundle-analyzer.
image

"lint": "eslint .",
"preview": "vite preview",
"prepack": "cp -f ../../LICENSE ./LICENSE"
"prepack": "cp -f ../../LICENSE ./LICENSE",
Copy link
Collaborator

Choose a reason for hiding this comment

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

../../LICENSE should be probably removed as well

on:
push:
tags:
- 'v*'
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not big fan of having "v" as prefix in tags.

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.

2 participants