Skip to content

chore: add Node version file#623

Merged
Aditya948351 merged 3 commits into
devpathindcommunity-india:masterfrom
nightcityblade:fix/issue-618
Jun 17, 2026
Merged

chore: add Node version file#623
Aditya948351 merged 3 commits into
devpathindcommunity-india:masterfrom
nightcityblade:fix/issue-618

Conversation

@nightcityblade

Copy link
Copy Markdown
Contributor

Description

Adds a root .nvmrc file to standardize the project Node.js version with the existing GitHub Actions lint workflow.

Fixes #618

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Local testing
  • Vercel Preview Deployment

Validated that .nvmrc contains 20 and matches .github/workflows/lint.yml (node-version: 20) with a local Node script.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have checked that the "DevPath India" branding remains intact

@github-actions github-actions Bot added good first issue gssoc26 This is a official GirlScript Summer of Code label. level:beginner Beginner level issues type:devops type:docs type:feature labels Jun 15, 2026
@Aditya948351

Copy link
Copy Markdown
Collaborator

@nightcityblade We cannot merge this, as the .nvmrc file content is Missing!

Do star the repo if not already!

@nightcityblade

Copy link
Copy Markdown
Contributor Author

Thanks — I pushed a follow-up commit to make the file explicit as .

@nightcityblade

Copy link
Copy Markdown
Contributor Author

Clean follow-up: I pushed a commit that makes .nvmrc explicit as 20.19.0.

@nightcityblade

Copy link
Copy Markdown
Contributor Author

Thanks — I rebased the branch and updated so the file now contains the explicit Node version ().

@nightcityblade

Copy link
Copy Markdown
Contributor Author

Thanks for the follow-up — I rechecked the PR branch and confirmed .nvmrc is present with the explicit content 20.19.0.

Validation run:

  • node -e "const fs=require('fs'); const v=fs.readFileSync('.nvmrc','utf8'); if(!/^20\\.19\\.0\\n?$/.test(v)){throw new Error('unexpected .nvmrc content: '+JSON.stringify(v))}; console.log('.nvmrc content OK:', JSON.stringify(v))"
  • git diff --check

git push origin fix/issue-618 reports Everything up-to-date, so no additional commit was needed beyond the existing .nvmrc fix commit.

@Aditya948351 Aditya948351 added gssoc:approved give 50+ base points and removed type:feature type:devops labels Jun 17, 2026
@Aditya948351 Aditya948351 merged commit 20c280c into devpathindcommunity-india:master Jun 17, 2026
@Aditya948351 Aditya948351 self-requested a review June 18, 2026 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issue gssoc:approved give 50+ base points gssoc26 This is a official GirlScript Summer of Code label. level:beginner Beginner level issues type:docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Config]: Add a .nvmrc file to standardize the Node version

2 participants