build(pci-common): add vite config to publish library - Githubissues

ovh / manager

OVHcloud Control Panel

https://ovh.github.io/manager/

BSD 3-Clause "New" or "Revised" License

216 stars 98 forks source link

build(pci-common): add vite config to publish library #14285

Open fredericvilcot opened 5 days ago

fredericvilcot commented 5 days ago

Question	Answer
Branch?	`master`
Bug fix?	no
New feature?	yes
Breaking change?	no
Tickets	Fix #TAPC-2321
License	BSD 3-Clause

[x] Try to keep pull requests small so they can be easily reviewed.
[x] Commits are signed-off
[ ] Only FR translations have been updated
[x] Branch is up-to-date with target branch
[x] Lint has passed locally
[x] Standalone app was ran and tested locally
[ ] Ticket reference is mentioned in linked commits (internal only)
[ ] Breaking change is mentioned in relevant commits

Description

This PR adds configuration to enable the publishing of the library.

Nota:

The build phase only targets esm file, because of the uselessness of building cjs or umd files that are not compatible anymore with certain libraries like @tanstak/react-query. Furthermore, all browsers are now compatible with ES modules so there is no need to weigh down the produced dist folder. As a consequence, there is no need to add output.globals instructions for rollup because this is only needed for umd files.
I added hidden sourcemap files to ensure a better debugging with tools like Sentry without exposing any reference to the source code in the browser.

Related

Will be treated with #14344

anooparveti commented 2 days ago

Rebasing with release branch.

sonarcloud[bot] commented 2 days ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud