Skip to content

Optimise CSS and JSON files#2569

Merged
PrestonN merged 1 commit intoFreeTubeApp:developmentfrom
absidue:optimise-css-json
Sep 13, 2022
Merged

Optimise CSS and JSON files#2569
PrestonN merged 1 commit intoFreeTubeApp:developmentfrom
absidue:optimise-css-json

Conversation

@absidue
Copy link
Member

@absidue absidue commented Sep 11, 2022


Optimise CSS and JSON files

Pull Request Type

  • Other - Output optimisation

Description
Optimise the CSS and JSON files during production builds. This unfortunately doen't make a big difference but I guess it's better than nothing. It is annoying that css-minimizer-webpack-plugin adds so many new development dependencies.

Screenshots (if appropriate)
dist directory:
dist_before
dist_after

app.asar:
resources_before
resources_after

Testing (for code that is not small enough to be easily understandable)
yarn dev and yarn build

Desktop (please complete the following information):

  • OS: Windows
  • OS Version: 10
  • FreeTube version: 0.17.1

@absidue absidue added PR: dependencies Pull requests that update a dependency file PR: waiting for review For PRs that are complete, tested, and ready for review labels Sep 11, 2022
@PrestonN PrestonN enabled auto-merge (squash) September 11, 2022 14:32
@github-actions github-actions bot added PR: merge conflicts / rebase needed and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels Sep 11, 2022
@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@absidue absidue added the PR: waiting for review For PRs that are complete, tested, and ready for review label Sep 11, 2022
PikachuEXE
PikachuEXE previously approved these changes Sep 12, 2022
@github-actions github-actions bot added PR: merge conflicts / rebase needed and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels Sep 12, 2022
@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@PrestonN PrestonN merged commit a69cbd3 into FreeTubeApp:development Sep 13, 2022
@absidue absidue deleted the optimise-css-json branch September 13, 2022 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants