Skip to content

Language branch - #193

Merged
matt-dahlgren merged 7 commits into
CA-Visualizer-for-Education:mainfrom
jayden-ong:language-branch
Jul 26, 2026
Merged

Language branch#193
matt-dahlgren merged 7 commits into
CA-Visualizer-for-Education:mainfrom
jayden-ong:language-branch

Conversation

@jayden-ong

@jayden-ong jayden-ong commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Overview

We want the ability to initialize projects in python, typescript, javascript, and java. We also want to be able to add use cases based on the language that was used when initializing the project.

Proposed Changes

Both initialization use cases now take in a "language" argument that creates a new project based on what language was passed in. If no language is passed in the cave command, it is default java. When creating use cases, the program will find the directory that is named after the language that was passed in and create the files accordingly.

...

Screenshots of your changes (if applicable)

How to Test & Review

(Help your reviewer by providing clear instructions on how to verify your changes. Include specific commands, edge cases to watch out for, or UI navigation paths.)

Type of Change

You can run npm run test or run the cave commands for initializing a new project and creating a new use case.

Type Applies?
🚨 Breaking change (fix or feature that would cause existing functionality to change) x
New feature (non-breaking change that adds functionality) x
🐛 Bug fix (non-breaking change that fixes an issue)
🎨 User interface change (change to user interface; provide screenshots)
♻️ Refactoring (internal change to codebase, without changing functionality)
🚦 Test update (change that only adds or modifies tests) x
📚 Documentation update (change that only updates documentation) x
📦 Dependency update (change that updates a dependency)
🔧 Internal (change that only affects developers or continuous integration)

Checklist

(Complete each of the following items for your pull request. Indicate that you have completed an item by changing the [ ] into a [x] in the raw text, or by clicking on the checkbox in the rendered description on GitHub.)

Before opening your pull request:

  • [x ] I have performed a self-review of my changes.

    • Check that all changed files included in this pull request are intentional changes.
    • Check that all changes are relevant to the purpose of this pull request, as described above.
  • [x ] I have added tests for my changes, if applicable.

    • This is required for all bug fixes and new features.
  • [x ] I have updated the project documentation, if applicable.

    • This is required for new features.
  • (Frontend) I have added/updated text in i18n JSON files for any new user-facing strings to support multilingual features.
    After opening your pull request:

  • [x ] I have verified that the CI tests have passed.

  • [ x] I have requested a review from a project maintainer, and a fellow student.

  • Technical Debt: If temporary workarounds or "TODOs" were used, I have opened a tracking issue to address them properly.

    • Linked Issues:

Questions and Comments

(Include any questions or comments you have regarding your changes.)

@jayden-ong
jayden-ong marked this pull request as ready for review July 22, 2026 17:04
@jayden-ong
jayden-ong requested a review from matt-dahlgren July 22, 2026 17:04

@matt-dahlgren matt-dahlgren left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

looks good to me thank you!

@matt-dahlgren
matt-dahlgren merged commit b5a9954 into CA-Visualizer-for-Education:main Jul 26, 2026
3 checks passed
@jayden-ong
jayden-ong deleted the language-branch branch August 1, 2026 17:15
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