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

Bug fix for checking delimited tables that define an object_length #687

Merged
merged 3 commits into from
Aug 23, 2023

Conversation

al-niessner
Copy link
Contributor

@al-niessner al-niessner commented Aug 22, 2023

🗒️ Summary

Unit tests are here but the actual fix is in NASA-PDS/pds4-jparser#97

⚙️ Test Data and/or Report

See automated unit tests

♻️ Related Issues

Closes #684

@al-niessner
Copy link
Contributor Author

@jordanpadams

jparser PR be merged and maven artifact updated for these unit tests to pass assuming I got all the cucumber stuff right.

@jordanpadams
Copy link
Member

Merged PR on pds4-jparser. Rerunning branch tests.

@jordanpadams jordanpadams changed the title 684: error when file size is given for delimited table file area Bug fix for checking delimited tables that define an object_length Aug 23, 2023
Copy link
Member

@nutjob4life nutjob4life left a comment

Choose a reason for hiding this comment

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

lgtm

@jordanpadams jordanpadams merged commit ec0283d into main Aug 23, 2023
@jordanpadams jordanpadams deleted the issue_684 branch August 23, 2023 21:07
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.

Validation failures are contingent on presence of <file_size> attribute in <File> class
3 participants