-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consolidate documentation/reports with coverage #194
Comments
@decoles are you sure you got the right issue number? It doesn't look like this issue has been resolved. |
@Anoushka444 @G1411 Opened a new branch consolidated_docs to collaborate on this issue |
@G1411 @Anoushka444 Please suggest which subtasks would suit you the best and start working on them! |
Sourabh and I'll work on the actions and readme |
Apologies I did not get the right number, thank you for fixing! |
#244 This PR has experimental changes for the Test and coverage report github actions.
|
In today's discussion with @Anoushka444 After some discussion, we decided to go with the second option. We created a package.json file (only to aid coverage-report workflow) in the root directory of the project and added the necessary dependencies. We also created a jest.config.js file, which lets us specify the test paths for both the frontend and backend. This configuration ensured that when we ran the tests, they would be read from both the frontend and backend folders, and the results would be combined to generate a single coverage report. By using this custom setup, we were able to have greater control over the coverage process and generate a single report that covered both the frontend and backend code. The coverage report is stored in the pantrynode/docs/coverage folder, making it easily accessible. The coverage folder (when generated) also features backend and frontend directories with html coverage of overall and specific tsx/ts files. We also experimented with running a single job for testing workflow to reduce the total number of pipelines (can be used if needed). We also corrected the backend test coverage , although we re still not sure how backend and their tests behave. |
Currently, there is a dedicated
backend/docs
folder for project documentation, but I would like a consolidated docs folder that includes any relevant documentation anywhere in the system. We need to do the followingbackend/docs
to adocs
folder in the root directorydocs
folder:README.md
updated) when it is accepted into main.The text was updated successfully, but these errors were encountered: