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

Testing static tool prettier #53

Open
wants to merge 4 commits into
base: f24
Choose a base branch
from
Open

Testing static tool prettier #53

wants to merge 4 commits into from

Conversation

bythehills
Copy link

@bythehills bythehills commented Oct 23, 2024 •
edited
Loading

What?

  • Installed and tested the prettier static analysis tool on our team repository.

Why?

  • Testing and research of prettier tool, a static tool that analyzes code syntax and style and formats it to be more readable.

Screenshots of Local Installation

Installing prettier

Screenshot 2024-10-23 at 12 25 04 AM

prettier config setup

Screenshot 2024-10-23 at 12 25 16 AM

Local prettier analysis

  • Ran prettier on all files
Screenshot 2024-10-23 at 12 26 11 AM image

Sample of prettier output on section of code

@bythehills bythehills self-assigned this Oct 23, 2024
@bythehills bythehills added the enhancement New feature or request label Oct 23, 2024
@a1inachen a1inachen added testing and removed enhancement New feature or request labels Oct 23, 2024
a1inachen
a1inachen approved these changes Oct 24, 2024
View reviewed changes
Copy link

@a1inachen a1inachen left a comment

Choose a reason for hiding this comment

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

very long haha, but makes sense looks good!

Michaelli8899
Michaelli8899 approved these changes Oct 24, 2024
View reviewed changes
Copy link

@Michaelli8899 Michaelli8899 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

see and installation, added to workflow file, see all the files changed

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

@Michaelli8899 Michaelli8899 Michaelli8899 approved these changes

@a1inachen a1inachen a1inachen approved these changes

@dominicteh1 dominicteh1 Awaiting requested review from dominicteh1

Assignees

@bythehills bythehills

Labels
testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bythehills @a1inachen @Michaelli8899