Epistimio / orion-dashboard

Other
0 stars 2 forks source link

[from epistimio] Add Github Actions and reformat code #4

Closed notoraptor closed 2 years ago

notoraptor commented 2 years ago

[ copied from #3 ]

Hi @bouthilx ! This is a pull request that adds Github actions to run Javascript and ReactJS testings. CI configuration is located in file .github/workflows/node.js.yml.

I added 2 example tests:

I also added to Github Actions CI a ci-check script provided by Carbon that checks file formatting, so I added a supplementary commit to fix code format.

What do you think?

bouthilx commented 2 years ago

Is it common practice to include the whole yarn-3.1.1.cjs file in the repository or should it be kept as an external dependency?

notoraptor commented 2 years ago

Is it common practice to include the whole yarn-3.1.1.cjs file in the repository or should it be kept as an external dependency?

Hi @bouthilx , it seems yarn suggests to include it: https://next.yarnpkg.com/getting-started/qa#which-files-should-be-gitignored

.yarn/releases is listed as a folder to keep, and they also say:

.yarn/plugins and .yarn/releases contain the Yarn releases used in the current repository (as defined by yarn set version). You will want to keep them versioned (this prevents potential issues if, say, two engineers use different Yarn versions with different features).

So, I guess we should keep it.

notoraptor commented 2 years ago

@bouthilx Updated with your suggestions

bouthilx commented 2 years ago

Sounds good, thanks for the clarification about yarn-3.1.1.cjs!

notoraptor commented 2 years ago

@bouthilx updated, now using queryByText+toBeInTheDocument instead of getByText. This should be more understandable.