Skip to content

Conversation

@matt-gadd
Copy link
Contributor

Type: bug
The following has been addressed in the PR:

  • All code has been formatted with prettier

Description:
The previous fix didn't actually work

@codecov
Copy link

codecov bot commented Mar 7, 2019

Codecov Report

Merging #253 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #253   +/-   ##
=======================================
  Coverage   52.17%   52.17%           
=======================================
  Files          10       10           
  Lines         621      621           
  Branches      156      156           
=======================================
  Hits          324      324           
  Misses        297      297
Impacted Files Coverage Δ
src/main.ts 88.63% <0%> (ø) ⬆️

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 f9f7cfc...87fceb4. Read the comment docs.

@matt-gadd matt-gadd merged commit 507e2fb into dojo:master Mar 7, 2019
agubler pushed a commit to agubler/cli-build-app that referenced this pull request Mar 8, 2019
* change to using req.accepts

* package-lock
agubler added a commit that referenced this pull request Mar 21, 2019
* Add slash to paths that need them in serve (#247)

* add slash to paths that need them

* only rewrite on html type requests

* fix test

* Change path rewriting to use req.accepts in serve mode (#253)

* change to using req.accepts

* package-lock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants