Skip to content

Issue 299 fixes#300

Merged
mmunin merged 2 commits into
mainfrom
issue-299-fixes
Aug 11, 2025
Merged

Issue 299 fixes#300
mmunin merged 2 commits into
mainfrom
issue-299-fixes

Conversation

@sumn2u

@sumn2u sumn2u commented Aug 11, 2025

Copy link
Copy Markdown
Owner

Typo and grammer fixes. Fixes #299

@vercel

vercel Bot commented Aug 11, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
learn-javascript ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 11, 2025 9:56am

This comment was marked as outdated.

@sumn2u sumn2u requested a review from Copilot August 11, 2025 09:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR addresses typo and grammar corrections in the file system documentation, fixing issue #299. The changes improve readability and accuracy of comments and documentation text.

  • Corrected spelling errors in comments and console output messages
  • Fixed grammatical issues and improved clarity of explanations
  • Updated synchronous writeFile example to remove incorrect callback parameter

Comment thread en/file-system/README.md
fs.writeFileSync("test.txt","mahima is good girl",() => {
console.log("This is sync: intentionally process is blocked ")
})
fs.writeFileSync("test.txt","mahima is good girl", 'utf8')

Copilot AI Aug 11, 2025

Copy link

Choose a reason for hiding this comment

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

The fs.writeFileSync function signature is incorrect. The encoding parameter should be passed as part of an options object: fs.writeFileSync("test.txt", "mahima is good girl", { encoding: 'utf8' }) or as the third parameter without quotes: fs.writeFileSync("test.txt", "mahima is good girl", 'utf8')

Copilot uses AI. Check for mistakes.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Done.

@mmunin

mmunin commented Aug 11, 2025

Copy link
Copy Markdown
Collaborator

Looks great, merging now

@mmunin mmunin merged commit f6fe5cc into main Aug 11, 2025
5 checks passed
@sumn2u sumn2u deleted the issue-299-fixes branch October 6, 2025 10:47
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.

Fix syntax errors in en/file-system/README.md

3 participants