Add right and bottom of area to JSON output by laigor · Pull Request #265 · tabulapdf/tabula-java
Navigation Menu
{{ message }}
tabulapdf / tabula-java Public
- Notifications You must be signed in to change notification settings
- Fork 449
Merged
Add right and bottom of area to JSON output#265
jazzido merged 1 commit intotabulapdf:masterfrom
laigor:master
Add right and bottom of area to JSON output#265
jazzido merged 1 commit intotabulapdf:masterfrom
laigor:master
Conversation
laigor
mentioned this pull request
Copy link
Contributor
jazzido
commented
Dec 7, 2018
jazzido commented
Dec 7, 2018Hi @laigor,
Thanks for your contribution! I noticed that the tests for the JSON writer are failing after your commit. Would you mind updating this PR with the fixes?
Thanks!
Copy link
Contributor Author
laigor
commented
Dec 9, 2018
laigor commented
Dec 9, 2018I saw that the tests do not pass, but I do not know how to fix them.
Copy link
Member
jeremybmerrill
commented
Dec 9, 2018
jeremybmerrill commented
Dec 9, 2018I haven't checked, but if I recall correctly, the test is just checking that the output hasn't changed. Since this adds the right and bottom... the output has changed. It's presumably just a matter of changing the file the tests check against to the current output...
jazzido
merged commit
545db99
into
tabulapdf:master
jazzido added a commit that referenced this pull request
Dec 11, 2018Copy link
Contributor
jazzido
commented
Dec 11, 2018
jazzido commented
Dec 11, 2018Tests modified, and PR merged. Thanks again @laigor !
EmpowerZ pushed a commit to EmpowerZ/tabula-java that referenced this pull request
Oct 23, 2020This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment