Skip to content

Add a generation error hierarchy - #19

Draft
mulkieran wants to merge 1 commit into
stratis-storage:masterfrom
mulkieran:master-generation-error
Draft

Add a generation error hierarchy#19
mulkieran wants to merge 1 commit into
stratis-storage:masterfrom
mulkieran:master-generation-error

Conversation

@mulkieran

@mulkieranmulkieran commented Nov 15, 2018

Copy link
Copy Markdown
Member

Related: #10.

@mulkieran

Copy link
Copy Markdown
MemberAuthor

Looks like we should be catching pyparsing's ParseBaseException. RecursiveGrammarException just doesn't apply here.

Signed-off-by: mulhern <amulhern@redhat.com>
@mulkieran
mulkieranforce-pushed the master-generation-error branch from 64ba4cb to c9891e2CompareNovember 15, 2018 15:41
@agroveragrover changed the title Add a generation error heirarchyAdd a generation error hierarchyNov 15, 2018
@mulkieran

Copy link
Copy Markdown
MemberAuthor

This PR is still partial, and the new exception isn't being used. It is only relevant if the D-Bus signature is ill-formed and can't be parsed. We can let this go until later.

@mulkieran
mulkieran marked this pull request as draft March 10, 2021 13:51
Sign up for freeto 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.

1 participant

@mulkieran