Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Automation Update #2619

Closed
wants to merge 20 commits into from
Closed

Automation Update #2619

wants to merge 20 commits into from

Conversation

MyDrift-user
Copy link
Contributor

@MyDrift-user MyDrift-user commented Aug 24, 2024 •
edited
Loading

Type of Change

  • Refactoring

Description

Improve Automation:

  • Instead of checking checkboxes and run tweaks run tweaks using config (remove UI dependency)
  • Don't generate UI on automation (improves speed)

TODO

  • run automation in synchronogical order (on automation)

Impact

Trying to massively improve automation without having to change the user's existing configuration.

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.

momcilovicluka
momcilovicluka reviewed Sep 22, 2024
View reviewed changes
scripts/main.ps1 Outdated Show resolved Hide resolved
@MyDrift-user @momcilovicluka
cleanup to app automation
a97f36f 
- use foreach instead of for loop
- remove unneeded redundancy

Co-authored-by: Luka Momčilović <l.momcilovic61@gmail.com>
@MyDrift-user MyDrift-user mentioned this pull request Oct 1, 2024
Draft
20 tasks
@MyDrift-user
Copy link
Contributor Author

closing this out due to inactive work. Just have to much to do rn, will revisit this as soon as possible :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@momcilovicluka momcilovicluka momcilovicluka left review comments

@ChrisTitusTech ChrisTitusTech Awaiting requested review from ChrisTitusTech ChrisTitusTech will be requested when the pull request is marked ready for review ChrisTitusTech is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MyDrift-user @momcilovicluka @og-mrk