Skip to content
This repository has been archived by the owner on Dec 15, 2023. It is now read-only.

Switch UI build system to pnpm #605

Merged
merged 14 commits into from
Aug 4, 2023
Merged

Switch UI build system to pnpm #605

merged 14 commits into from
Aug 4, 2023

Conversation

MindTooth
Copy link
Collaborator

@MindTooth MindTooth commented Aug 3, 2023

I've changed the setup to pnpm as more and more projects are doing the same. I've noted quite an increase in speed.

Also includes some other tweaks:

  • fixed legacy Tailwind option
  • inlined all resources
  • bump prettier
  • tweaked the documentation a bit (this needs a thorough rewrite)
  • moved types to regular deps (same package in same group)

Instruction for publishing

cd src/ui
pnpm install
pnpm run build:prod

Fixed #66.

@MindTooth MindTooth requested a review from tomrtk August 3, 2023 17:15
@MindTooth MindTooth self-assigned this Aug 3, 2023
@MindTooth MindTooth force-pushed the chore/switch_to_pnpm branch 3 times, most recently from 8869b24 to 916e78a Compare August 3, 2023 17:38
src/ui/README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@MindTooth MindTooth added done and removed doing labels Aug 4, 2023
@MindTooth MindTooth marked this pull request as ready for review August 4, 2023 18:23
@MindTooth MindTooth requested a review from tomrtk August 4, 2023 18:26
@MindTooth
Copy link
Collaborator Author

I think it's done now.

I'll work on the release pipeline next. But, first I'll see if I can tweak the running of Flask with watch or something. So it's easier to developer at the same time both Python and CSS/JS code in development mode.

Copy link
Owner

@tomrtk tomrtk left a comment

Choose a reason for hiding this comment

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

Ok, will you squeeze the commits? as some of them look to not be passing ci if checked out?

@MindTooth MindTooth merged commit 413bed8 into master Aug 4, 2023
9 checks passed
@MindTooth MindTooth deleted the chore/switch_to_pnpm branch August 4, 2023 19:02
@tomrtk tomrtk mentioned this pull request Aug 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document the use of npm in ui
2 participants