Skip to content

Conversation

jeevithakannan2
Copy link
Contributor

Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • Refactoring
  • Hotfix
  • Security patch
  • UI/UX improvement

Description

  • Refactor to use vhs github action

Testing

  • Tested on linutil fork.

Impact

  • The installation of the fonts take too much time. A workaround must be done for the fonts.

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no errors/warnings/merge conflicts.

@adamperkowski
Copy link
Collaborator

adamperkowski commented Oct 3, 2024

Wow this takes waaaay longer than the previous one.
https://github.com/jeevithakannan2/linutil/actions/runs/11156719955
Yeah the previous one was 1 minute 40-something seconds
This one is 4 minutes

@adamperkowski
Copy link
Collaborator

It installs a lot of unused fonts

@ghost
Copy link

ghost commented Oct 3, 2024

i like how the preview is navigating to the utilities tab when you can just search any option in any tab

@ghost
Copy link

ghost commented Oct 3, 2024

isnt system clean up in system setup

@adamperkowski
Copy link
Collaborator

i like how the preview is navigating to the utilities tab when you can just search any option in any tab

image

vhs action downloads jetbrains mono nf by default
@jeevithakannan2
Copy link
Contributor Author

Wow this takes waaaay longer than the previous one. https://github.com/jeevithakannan2/linutil/actions/runs/11156719955 Yeah the previous one was 1 minute 40-something seconds This one is 4 minutes

Now it takes only around 2min just like before.

@adamperkowski
Copy link
Collaborator

Now it takes only around 2min just like before.

Thanks :))

@ChrisTitusTech ChrisTitusTech merged commit e31eccd into ChrisTitusTech:main Oct 8, 2024
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