digitalr00ts / fizzbuzzx

An unnecessarily over-engineered fizzbuzz solution
Apache License 2.0
2 stars 2 forks source link

Basic Webpage Setup for Fizzbuzzx #39

Closed pelmini closed 3 years ago

codecov[bot] commented 3 years ago

Codecov Report

Merging #39 (a5fe9af) into master (0b300cc) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #39   +/-   ##
=======================================
  Coverage   53.33%   53.33%           
=======================================
  Files           1        1           
  Lines          45       45           
  Branches        2        2           
=======================================
  Hits           24       24           
  Misses         20       20           
  Partials        1        1           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0b300cc...0a2a5b5. Read the comment docs.

drts01 commented 3 years ago
pelmini commented 3 years ago
drts01 commented 3 years ago

I agree having a page to demo the API is a good nice-to-have.

I am not sure what this PR is adding ATM. Especially without a plan of how the frontend will talk to that backend. I think we should revisit later.

pelmini commented 3 years ago

Do you want me leave it open or should I close it?

drts01 commented 3 years ago

Lets close for now.