search

Bioinformatics-Research-Network / Bioinformed-Skill-App

Repository for the Bioinformed Skill Assessment App
https://skill.bioinformed.app
MIT License
4 stars 2 forks source link

edit padding of let's go box #52

Closed nghuixin closed 2 years ago

nghuixin commented 2 years ago

Code added in lines 527-530 to adjust border and padding of Let's Go box in profile.html

gitpod-io[bot] commented 2 years ago

codecov[bot] commented 2 years ago

Codecov Report

Merging #52 (b8c68f0) into main (d34aad3) will not change coverage. The diff coverage is 23.52%.

@@           Coverage Diff           @@
##             main      #52   +/-   ##
=======================================
  Coverage   52.65%   52.65%           
=======================================
  Files          20       20           
  Lines         773      773           
  Branches       70       70           
=======================================
  Hits          407      407           
  Misses        361      361           
  Partials        5        5
Flag Coverage Δ
webui 52.65% <23.52%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
webui/app/models/models.py 99.26% <ø> (ø)
webui/app/routes.py 41.50% <0.00%> (ø)
webui/app/utils/badgr_utils.py 15.58% <0.00%> (ø)
webui/app/utils/data_sync.py 13.55% <0.00%> (ø)
webui/tests/test_routes.py 100.00% <ø> (ø)
webui/app/__init__.py 83.87% <100.00%> (ø)
webui/app/crud/crud.py 26.00% <100.00%> (ø)
webui/app/utils/email_utils.py 58.82% <100.00%> (ø)
anmole17 commented 2 years ago

There are only formatting changes in /crud, this I think was done by black. That is not a problem, please wait until Henry gets time to for a proper review of the /webui. It is not preferred to make unnecessary changes in multiple folders as it will then lead to multiple pytest and coverage problems for your PR.

anmole17 commented 2 years ago

I think the issue was resolved in PR #57, so I am closing this PR, feel free to reopen if that is not the case.