Skip to content

Conversation

@yoptar
Copy link
Contributor

@yoptar yoptar commented Aug 18, 2023

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

In my environment self is not removed from the args dictionary, so I get an error of multiple self arguments in a constructor method.

Issue Number: N/A

After this fix reader and writer can be actually created

Other information

@yoptar yoptar changed the title fix: do not update the result of locals() directly topic fix: do not update the result of locals() directly Aug 18, 2023
@rekby rekby merged commit 1d525f6 into ydb-platform:main Aug 18, 2023
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