Feat: Testing and Linting

[CodeWithEmad]

This will add a Makefile, clean the code with isort and black, and fix typing issues.
Closes #151

[CodeWithEmad]

@regisb Should I modify the gh_refs_path function so that "refs": "GITHUB_API" + gh_refs_path() is replaced by "refs": gh_refs_path("openedx/frontend-app-account")?
Also, don't we wanna have make isort and make format run in the test.yml?

[regisb]

Should I modify the gh_refs_path function so that "refs": "GITHUB_API" + gh_refs_path() is replaced by "refs": gh_refs_path("openedx/frontend-app-account")?

Yes, good idea!

Also, don't we wanna have make isort and make format run in the test.yml?

I think that's convenient, yes.

[regisb]

I think that's a great step forward, and the PR looks good as it is. @arbrandes and @kdmccormick I'm curious to hear what you think?

[CodeWithEmad]

I think that's a great step forward, and the PR looks good as it is.

I'm happy to hear that!

black and isort steps added and gh_refs_path reformatted.

[CodeWithEmad]

I have resolved the conflicts that were caused by merging my last pull request.

Also I had another thing in mind. Shouldn't we update documents to inform contributors about running make ... commands before creating new PRs?

[CodeWithEmad]

Just a friendly reminder @kdmccormick @arbrandes

[kdmccormick]

Thanks for you patience @CodeWithEmad . I think this is a great improvement!

My only concern is that we should pin the development dependencies in order to keep the PR build stable--see my comment below.

[regisb]

Thanks a lot Emad! This is very useful. I'll let @arbrandes review, squash and merge.