Add Java and Node.js projects to dataset - #198
Conversation
|
@VikasPattar2006 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@VikasPattar2006 kindly resolve the conflicts before merging |
|
@komalharshita Resolved merge conflicts successfully, can you merge my PR? |
komalharshita
left a comment
There was a problem hiding this comment.
Thanks for contributing new Java and Node.js projects to the dataset — expanding language diversity is valuable for DevPath and the overall project structure is reasonably well organized.
However, there is one important blocking issue before this can be merged:
The dataset entries reference starter code files:
starter_code/library_management.javastarter_code/realtime_chat_app.js
but those files are not included in the PR. This creates broken references inside the application and compromises dataset integrity.
Please add the missing starter code files and verify:
- the JSON remains valid
- the projects render correctly in the UI
- recommendations still work as expected
Additional improvement suggestions:
- make roadmap steps more specific/educational
- consider using structured resource objects instead of plain strings for URLs in future dataset entries
Once the missing starter files are added and verified, this PR will be much closer to merge-ready.
|
Thanks for reviewing the PR and pointing this out. I’ve now added the missing starter code files:
I also verified:
I appreciate the additional suggestions regarding roadmap specificity and structured resource objects — I’ll keep those improvements in mind for future contributions. |
|
Thanks for reviewing the PR and pointing this out to me.
I've now added the missing starter code files.
…On Sun, 24 May 2026, 22:41 Komal Harshita, ***@***.***> wrote:
***@***.**** requested changes on this pull request.
Thanks for contributing new Java and Node.js projects to the dataset —
expanding language diversity is valuable for DevPath and the overall
project structure is reasonably well organized.
However, there is one important blocking issue before this can be merged:
The dataset entries reference starter code files:
- starter_code/library_management.java
- starter_code/realtime_chat_app.js
but those files are not included in the PR. This creates broken references
inside the application and compromises dataset integrity.
Please add the missing starter code files and verify:
- the JSON remains valid
- the projects render correctly in the UI
- recommendations still work as expected
Additional improvement suggestions:
- make roadmap steps more specific/educational
- consider using structured resource objects instead of plain strings
for URLs in future dataset entries
Once the missing starter files are added and verified, this PR will be
much closer to merge-ready.
—
Reply to this email directly, view it on GitHub
<#198 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BOLYM3RWJ3JS5WOHBLA35M344MUL5AVCNFSM6AAAAACZBCEM4OVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DGNJTGE2TONRSGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
@VikasPattar2006 kindly resolve the merge conflicts and ping me when you're done |
Summary
This PR expands the project dataset by adding beginner/intermediate backend-focused projects for Java and Node.js learners.
The newly added projects are:
Library Management System (Java)
Real-Time Chat Application (Node.js)
These additions improve technology diversity in the dataset and provide more backend-oriented learning options for beginners.
Related Issue
Closes #161
Type of Change
Data — adds new projects to
data/projects.jsonWhat Was Changed
data/projects.jsonHow to Test This PR
Checkout this branch:
Run the test suite:
Verify:
JSON loads successfully
new projects appear in recommendations
all required dataset fields are present
Expected output:
Test Results
Self-Review Checklist
I have read CONTRIBUTING.md and followed all guidelines
My branch name follows the convention
I have run
python tests/test_basic.pyand all tests passI have not introduced unrelated changes
All added dataset entries contain required JSON fields
Notes for Reviewer
This PR focuses on improving dataset diversity by adding backend-oriented beginner projects for Java and Node.js learners.