Skip to content

Header validation changes for Tind Spreadsheet validator#47

Open
davezuckerman wants to merge 2 commits into
mainfrom
AP-703-tind-validator
Open

Header validation changes for Tind Spreadsheet validator#47
davezuckerman wants to merge 2 commits into
mainfrom
AP-703-tind-validator

Conversation

@davezuckerman
Copy link
Copy Markdown
Contributor

No description provided.

@davezuckerman davezuckerman requested review from anarchivist and steve-sullivan and removed request for anarchivist May 29, 2026 18:28
Copy link
Copy Markdown
Member

@awilfox awilfox left a comment

Choose a reason for hiding this comment

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

r+wc - Per my understanding of the ticket, this seems to do the right thing. Would be more comfortable if someone more familiar with this tool also gave it a review, but from me, looks good.

Comment thread app/lib/tind_spread/tind_validation.rb Outdated
Copy link
Copy Markdown
Contributor

@steve-sullivan steve-sullivan left a comment

Choose a reason for hiding this comment

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

Looks good to me, granted, I'm not very familiar with the TIND validator

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.

3 participants