Dev#14
Closed
ApplebaumIan wants to merge 8 commits into
Closed
Conversation
…ll.java, add Warmup.java Co-authored-by: ApplebaumIan <9451941+ApplebaumIan@users.noreply.github.com>
Co-authored-by: ApplebaumIan <9451941+ApplebaumIan@users.noreply.github.com>
…-sh-button-functionality Support running arbitrary Java files via .run.sh and refactor Main.java to Rainfall.java
Member
Author
|
@copilot Please take the warmup assignment in Main.java and place it in Warmup.java. remove Rainfall.java from this branch. |
|
@ApplebaumIan I've opened a new pull request, #15, to work on those changes. Once the pull request is ready, I'll request review from you. |
Co-authored-by: ApplebaumIan <9451941+ApplebaumIan@users.noreply.github.com>
…a-debug-extension-again Remove Java debug extension from Codespace to isolate CLOVER run button
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request refactors the codebase to use
Rainfall.javaandRainfallTest.javainstead ofMain.javaandMainTest.java, improving naming consistency and flexibility in running Java files. It also introduces a newWarmup.javafile for a separate exercise and enhances the run script for more dynamic usage. The most important changes are grouped below.Refactoring and Naming Consistency
Main.javatoRainfall.javaand updated the main class name fromMaintoRainfallthroughout the codebase for clarity and consistency..clover-tests/MainTest.javato.clover-tests/RainfallTest.javaand updated all references fromMaintoRainfallin the test methods to match the new class name. [1] [2] [3] [4]Testing and Evaluation Script Updates
.clover-tests/run-tests.shto referenceRainfall.javaandRainfallTest.javainstead of the old file names, ensuring correct file checks and compilation for instructor evaluation. [1] [2]Run Script Improvements
.run.shto allow compiling and running any specified Java file, either via argument, environment variable, or by selecting the most recently modified.javafile. This improves flexibility for running different assignments..run.shto use the correct directory and class name, supporting the new dynamic file selection.New Exercise Addition
Warmup.javawith a basic revenue/expenses exercise template for students to complete.Configuration Update
clover.bugPercentageto.vscode/settings.jsonfor potential use in automated testing or grading.