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

Prettier Installation #43

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

Prettier Installation #43

wants to merge 2 commits into from

Conversation

kwarraich
Copy link
Collaborator

Description:
Successfully installed the Prettier tool

This tool is responsible for formatting code for easier readability. It allows for a full code base to easily be formatted into a consistant style. This tool was chosen to be installed so that way any new code written can be quickly formatted.

The current verison installed is 3.3.3
image

How to use Prettier:
To use the tool the following commands can be run to use the tool
npx prettier [YOUR FILE PATH] --write

Then to verify the tool ran and the files are formatted the following commnd can be run
npx prettier [YOUR FILE PATH] --check

Files Installed:
When installing prettier the following 2 files were created .prettierignore and .prettierrc and the following was in these files. I added in .prettierrc the additional parameters since we were getting issues when running npm run lint.

image

Testing:
We ran prettier on src/posts/category.js to format the file automatically.
Screenshot 2024-10-23 at 10 43 23 AM

After running we found no issues in funtionality as well

@kwarraich kwarraich added the Tools Installation installing tools needed to enhance development label Oct 23, 2024
@kwarraich kwarraich requested a review from rsharaf-dev October 23, 2024 12:40
@kwarraich kwarraich self-assigned this Oct 23, 2024
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11480072970

Details

  • 5 of 5 (100.0%) changed or added relevant lines in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.005%) to 82.687%

Files with Coverage Reduction New Missed Lines %
src/meta/errors.js 1 76.74%
Totals Coverage Status
Change from base Build 11442726769: -0.005%
Covered Lines: 22353
Relevant Lines: 25614

💛 - Coveralls

@kwarraich kwarraich requested a review from khalifat3 October 24, 2024 12:32
Copy link
Collaborator

@khalifat3 khalifat3 left a comment

Choose a reason for hiding this comment

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

Looks good, I can see that the prettier tool has been added to the file. It passes the test, and should be fine for future progress.

@kwarraich kwarraich removed the request for review from rsharaf-dev October 24, 2024 12:36
@Abdallah-Abdaljalil Abdallah-Abdaljalil added this to the Project 3A milestone Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tools Installation installing tools needed to enhance development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants