8000 Renamed input parameters to Leaf.make function to avoid shadowing by pdyraga · Pull Request #91 · keep-network/sortition-pools · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Renamed input parameters to Leaf.make function to avoid shadowing #91

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

Merged
merged 1 commit into from
Aug 25, 2020

Conversation

pdyraga
Copy link
Member
@pdyraga pdyraga commented Aug 25, 2020

Leaf library has operator, creationBlock, and weight functions and they were shadowed in make function by input parameters with the same name. Added underscores to parameter names to avoid shadowing.

Leaf library has operator, creationBlock, and weight functions and they
were shadowed in make function by input parameters with the same name.
Added underscores to parameter names to avoid shadowing.
@pdyraga pdyraga requested a review from nkuba August 25, 2020 08:56
@nkuba nkuba merged commit 677cbb3 into master Aug 25, 2020
@nkuba nkuba deleted the no-shadow branch August 25, 2020 09:01
@pdyraga pdyraga added this to the v1.1.1 milestone Aug 28, 2020
nkuba added a commit that referenced this pull request Aug 28, 2020
Bumping up version to 1.1.1 for a new release

The new release will contain no functionality changes but will re-organize the code based on Linter and Slither suggestions:
- Update linting configuration #87
- Small loop refactor to make Slither happier #90
- Renamed input parameters to Leaf.make function to avoid shadowing #91

See https://github.com/keep-network/sortition-pools/milestone/4?closed=1
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
0