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

Merge error code on archive steps #300

Closed
xml-project opened this issue Dec 19, 2019 · 2 comments · Fixed by #303
Closed

Merge error code on archive steps #300

xml-project opened this issue Dec 19, 2019 · 2 comments · Fixed by #303
Assignees

Comments

@xml-project
Copy link
Contributor

@xml-project xml-project commented Dec 19, 2019

@ndw and I had a longer discussion in the test suite repo about the precise situation to raise XC0081 or XC0085.

I therefor propose to change XC0085 to:

It is a dynamic error (err:XC0085) if the format of the archive does not match the specified format, cannot be understood, determined and/or processed.

XC0081 should be removed.

@xml-project xml-project self-assigned this Dec 19, 2019
@ndw

This comment has been minimized.

Copy link
Collaborator

@ndw ndw commented Dec 19, 2019

Agreed. Remove err:XC0081, adjust err:XC0085 as suggested.

@xml-project

This comment has been minimized.

Copy link
Contributor Author

@xml-project xml-project commented Dec 19, 2019

Will change tests too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.