Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: General maintenance of the repo #37

Closed
wants to merge 12 commits into from
Closed

fix: General maintenance of the repo #37

wants to merge 12 commits into from

Conversation

stickyPiston
Copy link
Member

@stickyPiston stickyPiston commented Mar 16, 2024 •
edited
Loading

closes #29
fixes #35

@stickyPiston
Enabled cors only in development
a6fa8d0
@stickyPiston
Fixed and moved tsc build to dist folder
e3fa08c
@stickyPiston
Added cors as dev dependency and completed package.json
f15f627
@stickyPiston
Updated dockerfile to work again
90a75d2
@stickyPiston
Created single docker compose file and added coverage reports for ci
0465022 
* The ci will now report if the coverage is below 85%
* The ci will now run prisma generate after reusing the cached node_modules
* The dockerfile is merged into one file, and the test and lint service have been removed. Additionally, migrations are run as part of the CMDs
@stickyPiston stickyPiston marked this pull request as draft March 17, 2024 13:23
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 17, 2024 •
edited
Loading

LCOV of commit 555a6ed during CI #109

Summary coverage rate:
  lines......: 94.1% (80 of 85 lines)
  functions..: 91.3% (21 of 23 functions)
  branches...: 92.9% (13 of 14 branches)

Files changed coverage rate:
                               |Lines       |Functions  |Branches    
  Filename                     |Rate     Num|Rate    Num|Rate     Num
  ===================================================================
  src/app.ts                   |82.6%     23|50.0%     2| 0.0%      1

@stickyPiston
Copy link
Member Author

stickyPiston commented Mar 17, 2024 •
edited
Loading

Summary of the things included in this PR:

@peppidesu peppidesu force-pushed the maintenance branch 2 times, most recently from 8dd1fe4 to 555a6ed Compare March 18, 2024 16:13
@peppidesu peppidesu deleted the maintenance branch May 2, 2024 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@stickyPiston @peppidesu