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.

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

Correcting description based on code #38797

Closed
wants to merge 1 commit into from

Conversation

siddtheone
Copy link

Since we aren't using <= or >= to compare hence boundary conditions are included.

Checklist:

  • I have read freeCodeCamp's contribution guidelines.
  • My pull request has a descriptive title (not a vague title like Update index.md)
  • My pull request targets the master branch of freeCodeCamp.
  • All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

Since we aren't using <= or >= to compare hence boundary conditions are included.
@gitpod-io
Copy link

gitpod-io bot commented May 12, 2020

@ShaunSHamilton
Copy link
Member

if (num > 10) {
  return "No";
}
if (num < 5) {
  return "No";
}
return "Yes";

will return "Yes" only if num is between 5 and 10 (5 and 10 included).

Follow the logic, by setting num = 5, then num = 10. What gets returned?

Now you tell me, for "Yes" to be returned, is num between 5 and 10 inclusive, or exclusive?

Close this issue, if you are satisfied.

@siddtheone siddtheone closed this May 12, 2020
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.

None yet

2 participants