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

Add Primitive data types article by Scaler #536

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ReddiRani8496
Copy link

Add article of primitive data types by Scaler.

@ReddiRani8496
Copy link
Author

Please review it. Thank you

README.md Outdated
@@ -163,6 +163,7 @@ Primitive values are directly assigned to a variable, and when you manipulate a
- [The differences between Object.freeze() vs Const in JavaScript — Bolaji Ayodeji](https://medium.com/@bolajiayodeji/the-differences-between-object-freeze-vs-const-in-javascript-4eacea534d7c)
- [Object to primitive conversion — JavaScript.info](https://javascript.info/object-toprimitive)
- [Methods of primitives - Javascript.info](https://javascript.info/primitives-methods)
- [What are the Primitive Data Types in JavaScript ? - Scaler](https://www.scaler.com/topics/what-are-the-primitive-data-types-in-javascript/)

Choose a reason for hiding this comment

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

Please correct the indentation.

@ReddiRani8496
Copy link
Author

Hi @Shaluag123 Line 165 has different indentation compared to 163 and 164. Which indentation I have to follow. Thank you

@ReddiRani8496
Copy link
Author

Previously I added accoring to 164, now I changed to indentation according to line 165. Can you please review and let me know, if there are any changes. Thank you.

@Shaluag123
Copy link

now it is fine no further modification needed.

@ReddiRani8496
Copy link
Author

Hi @leonardomso , can you please accept my Pull request. Thank you

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