Skip to content

Validator and Audit#336

Closed
COUNTERMetrics wants to merge 2 commits into
Project-Counter:plannedfrom
COUNTERMetrics:303-AuditandValidator
Closed

Validator and Audit#336
COUNTERMetrics wants to merge 2 commits into
Project-Counter:plannedfrom
COUNTERMetrics:303-AuditandValidator

Conversation

@COUNTERMetrics

Copy link
Copy Markdown
Contributor

Updated all instances of Validation Tool with COUNTER Validator. Added requirement in Appendix E for auditor to test COUNTER API as well as reports.

Updated all instances of Validation Tool with COUNTER Validator.
Added requirement in Appendix E for auditor to test COUNTER API as well as reports.
Added clarifications to audit timeline table 9.a.
Clarified fix period extensions.
Clarified that not planning dev time does not qualify for an extra extension.
@COUNTERMetrics COUNTERMetrics linked an issue Apr 12, 2026 that may be closed by this pull request
@berndoberknapp

Copy link
Copy Markdown
Member

We already have PR #323 for renaming COUNTER Validation Tool to COUNTER Validator. I would prefer to merge that PR and only merge the audit claricfications from this PR.

@COUNTERMetrics

Copy link
Copy Markdown
Contributor Author

We already have PR #323 for renaming COUNTER Validation Tool to COUNTER Validator. I would prefer to merge that PR and only merge the audit claricfications from this PR.

Does that mean you'd like me to remove those changes from this PR?

@berndoberknapp

Copy link
Copy Markdown
Member

Yes, please. You seem to have replaced "Validation Tool" with "COUNTER Validator" in some of the files, which has resulted in quite a number of "COUNTER COUNTER Validatior" which would have to be fixed. Additionally we should avoid mixing renaming things with other changes, which makes it easier to review relevant changes.

It might be easier to close the pull request and create a new one for the audit clarifications than trying to revert the renaming in this PR.

@berndoberknapp

Copy link
Copy Markdown
Member

Superseeded by #323 and #350.

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.

2 participants