cse210-works-on-my-machine / CSE210-Fall23-Team-3

Repo for Team 3 in CSE210 for the Fall 2023 quarter.
2 stars 0 forks source link

Rough draft team principles #44

Closed jacksonconte closed 9 months ago

jacksonconte commented 9 months ago

Reviewing this will act as signing off on it. Suggestions and feedback are highly encouraged.

netlify[bot] commented 9 months ago

Deploy Preview for fediblend ready!

Name Link
Latest commit 74de8a2c65760cbe4ed182e4ac723ca78bcea0e7
Latest deploy log https://app.netlify.com/sites/fediblend/deploys/6578bc21047f0800089bfa62
Deploy Preview https://deploy-preview-44--fediblend.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

lakshya-4gp commented 9 months ago

Principles look good, but in general this pull request is getting polluted by documentation metatdeta

TheCrossBoy commented 9 months ago

Principles look good, but in general this pull request is getting polluted by documentation metatdeta

yup, i fixed this on my branch

codecov[bot] commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (2d7da11) 92.26% compared to head (abafa42) 92.26%. Report is 1 commits behind head on main.

:exclamation: Current head abafa42 differs from pull request most recent head 74de8a2. Consider uploading reports for the commit 74de8a2 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #44 +/- ## ======================================= Coverage 92.26% 92.26% ======================================= Files 16 16 Lines 931 931 ======================================= Hits 859 859 Misses 72 72 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

omprakaash commented 9 months ago

LGTM !