-
Notifications
You must be signed in to change notification settings - Fork 20
Add Test and utility classes to create a jdt-ls workspace and test the commands #174
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
659b8d5
6555ff8
733cb79
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,4 @@ | ||
| { | ||
| "java.configuration.updateBuildConfiguration": "automatic" | ||
| "java.configuration.updateBuildConfiguration": "automatic", | ||
| "java.debug.settings.onBuildFailureProceed": true | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,7 @@ | ||
| <?xml version="1.0" encoding="UTF-8"?> | ||
| <classpath> | ||
| <classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-11"/> | ||
| <!--<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-17"/>--> | ||
| <classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER"/> | ||
|
Comment on lines
+3
to
+4
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chainClarify Java version strategy and remove commented code. The change from a pinned JavaSE-17 container to a generic
Please clarify the intended Java version strategy: if JavaSE-17 is required (as the manifest suggests), restore the pinned container; otherwise, update the manifest to match the generic approach. - <!--<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-17"/>-->
- <classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER"/>
+ <classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-17"/>I'll verify the review comment by examining the relevant configuration files. 🏁 Script executed: #!/bin/bash
# Search for MANIFEST.MF files in the repository, particularly in the test bundle
fd -t f "MANIFEST.MF" | head -20Length of output: 93 🏁 Script executed: #!/bin/bash
# Find and examine the .classpath file being reviewed
find . -path "*/java-analyzer-bundle.test/.classpath" -type fLength of output: 38 🏁 Script executed: #!/bin/bash
# Look at the .classpath file content to confirm current state
if [ -f "java-analyzer-bundle.test/.classpath" ]; then
cat "java-analyzer-bundle.test/.classpath"
fiLength of output: 564 🏁 Script executed: #!/bin/bash
# Search for MANIFEST.MF in or near the java-analyzer-bundle.test directory
fd -t f "MANIFEST.MF" | xargs -I {} sh -c 'echo "=== {} ===" && head -20 "{}"'Length of output: 1107 Remove commented code and align .classpath with manifest Java version requirement. Verification confirms the review concerns are legitimate:
Either restore the pinned JavaSE-17 container on line 4 to match the manifest, or update the manifest to reflect the generic approach. Remove the commented line regardless. 🤖 Prompt for AI Agents
jmle marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| <classpathentry kind="con" path="org.eclipse.pde.core.requiredPlugins"/> | ||
| <classpathentry kind="src" path="src/main/java/"> | ||
| <attributes> | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| <?xml version="1.0" encoding="UTF-8"?> | ||
| <projectDescription> | ||
| <name>springboot-todo</name> | ||
| <comment></comment> | ||
| <projects> | ||
| </projects> | ||
| <buildSpec> | ||
| <buildCommand> | ||
| <name>org.eclipse.jdt.core.javabuilder</name> | ||
| <arguments> | ||
| </arguments> | ||
| </buildCommand> | ||
| <buildCommand> | ||
| <name>org.eclipse.m2e.core.maven2Builder</name> | ||
| <arguments> | ||
| </arguments> | ||
| </buildCommand> | ||
| </buildSpec> | ||
| <natures> | ||
| <nature>org.eclipse.jdt.core.javanature</nature> | ||
| <nature>org.eclipse.m2e.core.maven2Nature</nature> | ||
| </natures> | ||
| </projectDescription> |
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,118 @@ | ||||||
| # Awesome To-Do App | ||||||
|
|
||||||
| ## Overview | ||||||
|
|
||||||
| **Awesome To-Do App** is a versatile task management application tailored to assist users in organizing tasks, managing deadlines, and prioritizing activities. The app offers an intuitive user interface complete with a spectrum of features including task creation, deletion, pagination, and advanced error handling. | ||||||
|
|
||||||
| In this README, you'll find a comprehensive guide that details your project, enumerates its key functionalities, lists the technologies employed, provides installation instructions, and articulates usage instructions. | ||||||
|
|
||||||
|  | ||||||
|
|
||||||
|  | ||||||
|
|
||||||
| ## Features | ||||||
|
|
||||||
| ### Task Management | ||||||
|
|
||||||
| - Create and manage tasks with essential details such as titles, descriptions, and due dates. | ||||||
| - Tasks are elegantly presented in card format, enhancing visibility and comprehension. | ||||||
| - Effortlessly delete tasks with permanent removal from the application. | ||||||
|
|
||||||
| ### Pagination | ||||||
|
|
||||||
| - Enhance user experience by paginating tasks, ensuring a clutter-free view. | ||||||
| - Navigate seamlessly through the task list with "Previous" and "Next" buttons for effortless organization. | ||||||
|
|
||||||
| ### Error Handling | ||||||
|
|
||||||
| - Robust error handling, encompassing gracefully displayed custom error pages and user-friendly messages. | ||||||
| - Guard against requests that seek pages beyond the total available count, offering a polished and secure user experience. | ||||||
|
|
||||||
| ### Advanced Features | ||||||
|
|
||||||
| - Responsive design adapting to diverse devices, guaranteeing a harmonious experience on any platform. | ||||||
| - Intuitive pop-up modals for confirming task deletion, enriching user interaction. | ||||||
|
|
||||||
| ## Technologies Used | ||||||
|
|
||||||
| **Frontend:** | ||||||
|
|
||||||
| - HTML | ||||||
| - Thymeleaf (for server-side rendering) | ||||||
| - JavaScript | ||||||
| - jQuery | ||||||
| - Bootstrap (for styling and modals) | ||||||
|
|
||||||
| **Backend:** | ||||||
|
|
||||||
| - Spring Boot (Java-based framework) | ||||||
| - Spring MVC | ||||||
| - Spring Data JPA (for database access) | ||||||
| - MySQL (as the database) | ||||||
|
|
||||||
| ## Installation | ||||||
|
|
||||||
| 1. **Clone the Repository:** | ||||||
|
|
||||||
| ```bash | ||||||
| git clone https://github.com/adampeer/spring-boot-todo-app.git | ||||||
| cd awesome-todo-app | ||||||
| ``` | ||||||
|
|
||||||
| 2. **Database Configuration:** | ||||||
|
|
||||||
| - Install MySQL and create a database. | ||||||
| - Update the `application.properties` file with your database jdtLSConfiguration such as username, password, database name and port number. | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix typo in documentation. The word "jdtLSConfiguration" should be "configuration". This appears to be a variable name accidentally left in the documentation. Apply this diff: - - Update the `application.properties` file with your database jdtLSConfiguration such as username, password, database name and port number.
+ - Update the `application.properties` file with your database configuration such as username, password, database name and port number.📝 Committable suggestion
Suggested change
🧰 Tools🪛 LanguageTool[grammar] ~65-~65: Ensure spelling is correct (QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1) 🤖 Prompt for AI Agents |
||||||
|
|
||||||
| 3. **Build and Run the Application:** | ||||||
|
|
||||||
| ```bash | ||||||
| ./mvnw clean package | ||||||
| java -jar target/awesome-todo-app-0.1.jar | ||||||
| ``` | ||||||
|
|
||||||
| 4. **Access the Application:** | ||||||
|
|
||||||
| Open a web browser and go to `http://localhost:8080` or whatever port you've set in application.properties file. | ||||||
|
|
||||||
| ## Usage | ||||||
|
|
||||||
| 1. **Create a Task:** | ||||||
|
|
||||||
| - Fill out the task creation form, providing a title, description, and due date. | ||||||
| - Click the "Create Task" button. | ||||||
|
|
||||||
| 2. **Pagination:** | ||||||
|
|
||||||
| - Use the "Previous" and "Next" buttons to navigate through your task list. | ||||||
| - Each page typically displays 6 tasks. | ||||||
|
|
||||||
| 3. **Delete a Task:** | ||||||
|
|
||||||
| - Each task card includes a "Delete" button. | ||||||
| - Click the "Delete" button to trigger a confirmation modal. | ||||||
| - Confirm the task deletion by clicking "Yes" in the modal. | ||||||
|
|
||||||
| 4. **Error Handling:** | ||||||
|
|
||||||
| - Error pages and messages are displayed for various error scenarios. | ||||||
| - Friendly error messages are shown to users. | ||||||
|
|
||||||
| 5. **Advanced Features:** | ||||||
|
|
||||||
| - Responsive design ensures a seamless experience on different devices. | ||||||
| - Confirmation modal for task deletion adds a layer of user interaction. | ||||||
|
|
||||||
| ## Feedback and Support | ||||||
|
|
||||||
| We welcome your feedback and suggestions. If you encounter any issues or have ideas for improvements, please open an issue on our GitHub repository. | ||||||
|
|
||||||
| ## License | ||||||
|
|
||||||
| This project is licensed under the MIT License. Feel free to use it, modify it, and share it as you see fit. | ||||||
|
|
||||||
| ## Author | ||||||
|
|
||||||
| - [Adam Peer](https://github.com/adampeer) | ||||||
|
|
||||||
| --- | ||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix incorrect location code in comment.
The comment shows
"package": 1but based on the switch statement (line 191), package location should be11, not1. Location1is used for "inheritance" (line 178).📝 Committable suggestion
🤖 Prompt for AI Agents