Skip to content

Fix VariableTypes solution errors - #27

Merged
phrdang merged 3 commits into
masterfrom
fix-pr-22
Jul 7, 2021
Merged

Fix VariableTypes solution errors#27
phrdang merged 3 commits into
masterfrom
fix-pr-22

Conversation

@phrdang

Copy link
Copy Markdown
Member

Affected Sections

  • Affects Ch. 2.1 What is a string?

Changes

  • Rename duplicate decimal variable
  • Remove unnecessary d in the number double
  • Fix type of dogsOut and name variables
  • Suppress unused variable warnings

Notes

- Rename duplicate `decimal` variable
- Remove unnecessary `d` in the `number` double
- Fix type of `dogsOut` and `name` variables
- Suppress unused variable warnings
@phrdang
phrdang requested review from atijmahesh and removed request for AlveeraMunshi and JJ27July 1, 2021 20:20
JJ27
JJ27 approved these changes Jul 7, 2021
@phrdang
phrdang removed the request for review from atijmaheshJuly 7, 2021 18:09
@phrdang
phrdang merged commit 0b3b773 into masterJul 7, 2021
@phrdang
phrdang deleted the fix-pr-22 branch July 7, 2021 18:11
@phrdang

Copy link
Copy Markdown
MemberAuthor

Added to Notion as a practice problem in 2.1

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.

2 participants

@phrdang@JJ27