Skip to content

Conversation

@corona10
Copy link
Member

@corona10corona10 commented Jun 11, 2020

Copy link
Member

@vstinnervstinner left a comment

Choose a reason for hiding this comment

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

I would prefer to start with a PR which adds a module state to get NisError.

Otherwise, it's unclear who owns this exception.

@bedevere-bot
Copy link

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@corona10
Copy link
MemberAuthor

I would prefer to start with a PR which adds a module state to get NisError.

Good! I will submit the PR for that

@corona10corona10 deleted the bpo-1635741-nis branch February 15, 2024 01:41
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@corona10@bedevere-bot@vstinner@the-knights-who-say-ni