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.

Sign up for GitHub

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

Jump to bottom

Fix #11 Update structs.rs to use function parameter, fixed value #10

Merged
merged 1 commit into from
Apr 18, 2021
Merged

Fix #11 Update structs.rs to use function parameter, fixed value #10

merged 1 commit into from
Apr 18, 2021

Conversation

adamvm
Copy link
Contributor

@adamvm adamvm commented Apr 17, 2021 •
edited
Loading

Function parameter unused, value inside function hardcoded

@adamvm adamvm changed the title Update structs.rs to use function parameter, previously fixed value h… Update structs.rs to use function parameter, previously fixed value Apr 17, 2021
@adamvm adamvm mentioned this pull request Apr 18, 2021
Closed
@adamvm
Copy link
Contributor Author

adamvm commented Apr 18, 2021 •
edited
Loading

Linked issue: #11

@adamvm adamvm changed the title Update structs.rs to use function parameter, previously fixed value Fix #11 Update structs.rs to use function parameter, fixed value Apr 18, 2021
creativcoder
creativcoder approved these changes Apr 18, 2021
View reviewed changes
Copy link
Collaborator

@creativcoder creativcoder left a comment

Choose a reason for hiding this comment

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

Hey @adamvm Thank you for the fix!

@creativcoder creativcoder merged commit bd100c7 into PacktPublishing:master Apr 18, 2021
@adamvm adamvm deleted the adamvm-patch-1 branch June 4, 2021 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@creativcoder creativcoder creativcoder approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@adamvm @creativcoder