Skip to content
This repository has been archived by the owner on Aug 1, 2021. It is now read-only.

update the vendorized TableRex and namespace it with Licensir #23

Merged
merged 1 commit into from
Jun 7, 2021
Merged

update the vendorized TableRex and namespace it with Licensir #23

merged 1 commit into from
Jun 7, 2021

Conversation

aseigo
Copy link
Contributor

@aseigo aseigo commented Jun 7, 2021

Besides updating the TableRex code, this allows Licensir to be used with other libraries that use TableRex as a dependency such as ecto_psql_extras

besides updating the TableRex code, this allows Licensir to be
used with other libraries that use TableRex as a dependency
@coveralls
Copy link

coveralls commented Jun 7, 2021

Pull Request Test Coverage Report for Build 63

  • 15 of 26 (57.69%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.4%) to 64.311%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/table_rex/renderer/text.ex 11 12 91.67%
lib/table_rex/table.ex 0 3 0.0%
lib/table_rex/cell.ex 4 11 36.36%
Totals Coverage Status
Change from base Build 62: -0.4%
Covered Lines: 182
Relevant Lines: 283

💛 - Coveralls

1 similar comment
@coveralls
Copy link

Pull Request Test Coverage Report for Build 63

  • 15 of 26 (57.69%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.4%) to 64.311%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/table_rex/renderer/text.ex 11 12 91.67%
lib/table_rex/table.ex 0 3 0.0%
lib/table_rex/cell.ex 4 11 36.36%
Totals Coverage Status
Change from base Build 62: -0.4%
Covered Lines: 182
Relevant Lines: 283

💛 - Coveralls

Copy link
Owner

@unnawut unnawut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@unnawut unnawut merged commit a4dc1c3 into unnawut:master Jun 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants