DRAFT [Issue#3] add tools for static code analysis - Prettier init config old

[KarolinaZakrzewska]

Partially Resolves #3

Scope of changes

• added the use of single quotes (') in Prettier configuration
• added appropriate line break depending on the system, for Windows: CRLF (\r\n)
• Prettier is set as the default formatting
• formatting of all files corrected

How to use

• install Prettier

npm install --save-dev --save-exact prettier

• add an exclusion file .prettierignore

package-lock.json
playwright-report

• add the rules file .prettierrc.json

{
  "singleQuote": true,
  "endOfLine": "auto"
}

• run Prettier formatting

npx prettier --write .

Additionally, you can add the VSC extension: Prettier - Code formatter (you see it in extensions as recommended)

• set as default formatting
  Right-click in the file e.g. package.json -> Format Document With... -> Configure Default Formatter... -> Select Prettier

[kat-kan]

Is PR title correct?

[bugITwhisperer]

@KarolinaZakrzewska : Congrats on your first PR!!! 😸 🎉
As discussed on Discord, please install the prettier lib via npm package manager 🙏🏻

[bugITwhisperer]

Is PR title correct?

Well, it didn't say much about the changes, so I've just adjusted it as per our discussion on Discord - to keep the current convention going:

[Issue#{number}] {short description of changes in present tense with action and the subject on which the action was made)
where:

• number is the referenced Issue that was created to cover the change from PR

short description in present tense

• with action} describes the action: add/create/update/fix etc.
• and the subject on which the action was made: README file, Playwright config, and such

Anyways, let's discuss the approach in more details in a upcoming meeting. Coz we might decide on something else instead. It's just how I do it for now, but my approach has been also evolving a bit since I started😅

[KarolinaZakrzewska]

Is PR title correct?

now it's ok, thank you @bugITwhisperer

[kat-kan]

@KarolinaZakrzewska please finish the changes - as I remember you're still working on it :)

[KarolinaZakrzewska]

closing due to conflicts with the latest version of main

[bugITwhisperer]

@KarolinaZakrzewska : are you planning to open a new PR with the Prettier config changes? :)

[KarolinaZakrzewska]

@KarolinaZakrzewska : are you planning to open a new PR with the Prettier config changes? :)

Yes, I prepared a new PR #37