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.

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

chore(script): fix assistant.sh script #1777

Merged
merged 1 commit into from
Jan 30, 2024
Merged

chore(script): fix assistant.sh script #1777

merged 1 commit into from
Jan 30, 2024

Conversation

stavares843
Copy link
Member

@stavares843 stavares843 commented Jan 30, 2024

What this PR does 📖

in dev, would only have option 1, 2 and 3 and would not install gum if wasn't already installed

Captura de ecrã 2024-01-30, às 01 02 36

and the 2) would say the following

image

in this branch, if you don't have gum installed, will install automatically using brew

Gravacao.do.ecra.2024-01-30.as.00.15.23.mov

then you have the following options

  1. it shows 3 options and then selecting each one it tells you the extensions command
    in this case it doesnt really execute just writes the command for the user to use 😂
  2. executes cargo run
  3. list of each option in more detail
  4. to run clippy
  5. to run fmt
  6. to auto fix clippy
  7. to auto fix fmt
fix_script001.mov

if you have changes locally will warn you about those when running the auto fix linter commands

Captura de ecrã 2024-01-30, às 00 49 51

@github-actions github-actions bot added Don't merge yet DO NOT MERGE Missing dev review Still needs to be reviewed by a dev labels Jan 30, 2024
@stavares843 stavares843 merged commit eaf1ff8 into dev Jan 30, 2024
10 of 24 checks passed
@stavares843 stavares843 deleted the sara/fix-script branch January 30, 2024 01:19
@github-actions github-actions bot removed Missing dev review Still needs to be reviewed by a dev Don't merge yet DO NOT MERGE labels Jan 30, 2024
Copy link
Contributor

UI Automated Tests execution is complete! You can find the test results report here

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.

2 participants