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.

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

feat(routes): add doc-to-txt route #1445

Merged
merged 3 commits into from
May 8, 2023
Merged

feat(routes): add doc-to-txt route #1445

merged 3 commits into from
May 8, 2023

Conversation

Fdawgs
Copy link
Owner

@Fdawgs Fdawgs commented May 3, 2023

closes #254

Todo:

  • Check CFB is a doc file as the file-type dep doesn't handle them

@Fdawgs Fdawgs self-assigned this May 3, 2023
@Fdawgs Fdawgs force-pushed the feat/routes/doc branch from 6d98005 to eab53f4 Compare May 3, 2023 18:16
@Fdawgs Fdawgs changed the title feat(routes): add /doc/txt conversion route feat(routes): add doc-to-txt route May 3, 2023
@Fdawgs Fdawgs marked this pull request as draft May 4, 2023 07:13
@Fdawgs Fdawgs force-pushed the feat/routes/doc branch from eab53f4 to 1b63166 Compare May 8, 2023 10:00
@Fdawgs Fdawgs marked this pull request as ready for review May 8, 2023 11:36
@coveralls
Copy link

coveralls commented May 8, 2023

Pull Request Test Coverage Report for Build 4915778803

  • 39 of 39 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 4913736003: 0.0%
Covered Lines: 611
Relevant Lines: 611

💛 - Coveralls

@Fdawgs Fdawgs merged commit 66a7739 into main May 8, 2023
@Fdawgs Fdawgs deleted the feat/routes/doc branch May 8, 2023 13:54
@coveralls
Copy link

Pull Request Test Coverage Report for Build 4914666720

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 38 of 39 (97.44%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.3%) to 99.741%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/routes/doc/txt/index.js 21 22 95.45%
Totals Coverage Status
Change from base Build 4913736003: -0.3%
Covered Lines: 568
Relevant Lines: 569

💛 - Coveralls

1 similar comment
@coveralls
Copy link

Pull Request Test Coverage Report for Build 4914666720

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 38 of 39 (97.44%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.3%) to 99.741%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/routes/doc/txt/index.js 21 22 95.45%
Totals Coverage Status
Change from base Build 4913736003: -0.3%
Covered Lines: 568
Relevant Lines: 569

💛 - Coveralls

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.

Add DOC to TXT route
2 participants