Ruike enforce no space in file name

[StrawberryCalpico]

Description

Remove all files with filename contains a white space, and enforce rule by setting .gitignore

Related PRS (if any):

N/A

Main changes explained:

Changed several files' filename, replace ' ' with '_'
add **\ ** in gitignore to ignore changes with whitespace in the filename

How to test:

1. check into current branch
2. do npm install (optional)
3. go to the parent directory of frontend/backend folder, for example, the Github folder
4. run the following python file and make sure no file path was printed

import os

backend = "HGNRest"
frontend = "HighestGoodNetworkApp"

def checkfile(folderName):
        for fileName in os.scandir(folderName):
                if os.path.isfile(fileName) and ' ' in os.path.basename(fileName):
                        print(os.path.relpath(fileName))
                elif os.path.isdir(fileName):
                        checkfile(fileName)

checkfile(backend)
checkfile(frontend)

5. can also test by creating new file with filename contains ' ', check that there's no change in git

Screenshots or videos of changes:

add a new file that contains whitespace in filename

i was using github desktop, make sure there's no local change

Note:

Make sure when run the script, the frontend/backend folder are in the same directory
such as
|| /Github
|| => *.py(replace * with any name)
|| => /frontend
|| => /backend
and run python3 *.py(replace * with any name)

[sxiong5]

Hey @StrawberryCalpico, I have run the Python, and everything looks good. Nice job!

[tsunami776]

Hi @StrawberryCalpico, I ran the python test you provided for dev and your branch. I found files that not in node_modules now do not have space in their file names. One minor question I wondered is if you can skip checking files in node_modules and .git. Nice work overall!
dev:

Your branch:

[ChirayuRai]

I checked out your PR, and it looks good! It only shows one space in node_modules in the backend folder. However, I don't know if that's important and should be something to consider? In the future it might be better to just skip over the folders in the gitignore for each directory or something.

[Iris-Song]

HI, I have tested your PR. It works as intended. Good job.

[yaow62023]

I have tested your pr, run the python code. No error at this pr.

[Shubhu0]

Hi @StrawberryCalpico Tested and reviewed your PR and it is working as intended.

[zuhang2000]

Hi, I've tested your PR following your instructions. The code passes without a error. It reports the results as expected. Good job.

[Aishwaryak01]

Hi Ruike,
Went through the regex provided in the .gitignore file to remove the space from the filename. It looks good to me. Approved!

[one-community]

Thank you all, merging!

[ManeeshBuddha21]

Tested alongside Frontend PR #3647, the frontend returns the “Failed to post to LinkedIn” error.

[Vinay944924]

Tested the PR locally and have provided the screenshot in Frontend PR-3647

[Vikas-8055]

Hi, I've tested your PR. The code passes without a error.

[Shravan-neelamsetty]

Hi, I've tested your PR. The code passes.