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

fix: use multiple conditional (ternary) operators #38990

Closed
wants to merge 1 commit into from
Closed

fix: use multiple conditional (ternary) operators #38990

wants to merge 1 commit into from

Conversation

niteshseram
Copy link
Contributor

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 #38872

Changed comment for more clarification

@gitpod-io
Copy link

gitpod-io bot commented Jun 2, 2020

@ShaunSHamilton
Copy link
Member

@niteshseram , I am closing this PR, due to the fact that you are not a first-time contributor. You are welcome to work on any other issues without the first-timers welcome label, and with the help wanted label.

@niteshseram
Copy link
Contributor Author

@SKY020 Okay my bad. I thought even I can make the PR.

@niteshseram niteshseram deleted the fix/use-multiple-ternary-operators branch June 2, 2020 13:54
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.

Bug in Challenge: Use Multiple Conditional (Ternary) Operators.
2 participants