multiverseweb / Dataverse

This software is currently under development. It can be used to visualise data into various forms of plots ranging from primary to advanced level. It can also be used as a finance tracker.
https://multiverse-dataverse.netlify.app
MIT License
39 stars 109 forks source link

[Website] Proposal to add Prettier to the project. #321

Open amin-xiv opened 2 weeks ago

amin-xiv commented 2 weeks ago

Here are the values: printWidth : (doubt: it should be the number of characters per line right?) tabWidth : 4 useTabs : true bracketSpacing : false arrowPens : true trailingComma : false

amin-xiv commented 1 week ago

@multiverseweb Any thoughts about this? 😊

multiverseweb commented 1 week ago

Okay @amin-xiv, make sure it doesn't affect the functionality of the code. Also, will it run for all files or only for python?

multiverseweb commented 1 week ago

Here are the values: printWidth : (doubt: it should be the number of characters per line right?) tabWidth : 4 useTabs : true bracketSpacing : false arrowPens : true trailingComma : false

@amin-xiv I accidentally edited your comment instead of quoting a reply 🥲

amin-xiv commented 1 week ago

@multiverseweb 1/No it doesn't affect the functionality. 2/Prettier doesn't work for python, it works for JS, TS, CSS, HTML, markdown, YAML and JSON. It simply arranges the code, according to your configuration, you can define the variables that was stated above. Thank you.

amin-xiv commented 1 week ago

@multiverseweb And yes printWidth, are the characters per line that require the extension to automatically wrap the rest of the text to the next line.

multiverseweb commented 1 week ago

@multiverseweb And yes printWidth, are the characters per line that require the extension to automatically wrap the rest of the text to the next line.

Okay @amin-xiv go ahead. Set the appropriate value of printWidth yourself, assigned.

amin-xiv commented 1 week ago

@multiverseweb Thank you very much sir.

amin-xiv commented 1 week ago

@multiverseweb Sorry, but what about singleQuote , if it is set to true, it will change every double quote to single, and when it is false, it will do the opposite. Note: this setting doesn't affect template strings or HTML attributes or JSON strings.

multiverseweb commented 1 week ago

@multiverseweb Sorry, but what about singleQuote , if it is set to true, it will change every double quote to singe, and when it is false, it will do the opposite. Note: this setting doesn't affect template strings or HTML attributes.

Set it to false.

amin-xiv commented 1 week ago

@multiverseweb So far I have added it to the project, what is only remaining is to update the README.md file.

multiverseweb commented 1 week ago

@amin-xiv nice, I would like to know what changes are required in the README.md file?

amin-xiv commented 1 week ago

@multiverseweb In order to allow prettier: 1/The prettier extension must be installed, so that will be included. 2/In order for it to work, correctly, with the correct version and settings, the node_modules/ folder has to be installed, so I will just add after you clone the repository in the README file, simply run npm install which is going to install all the dependencies required for it to work across al users. I added the node_modules/ to the .gitignore folder because it can add unnecessary space, and the user only needs the package.json and the package-lock.json folders in order to install it, so they will be included.

In conclusion, what will be added to the README file are going to be some few additions in the section of Prerequisites in order to set it up correctly.

multiverseweb commented 1 week ago

Okay @amin-xiv. In a few days from now I'll be fixing the structure of the repo and automating the process of project set-up. For this, I'll add a file requirements.txt which will include all the modules that need to be installed before using the software. You can continue with the steps you've mentioned.

amin-xiv commented 1 week ago

@multiverseweb Okay, Thanks.