Skip to content

Merge Eclipse projects and clarify README#28

Open
PhilWakelin wants to merge 11 commits intomainfrom
piw-readme-update
Open

Merge Eclipse projects and clarify README#28
PhilWakelin wants to merge 11 commits intomainfrom
piw-readme-update

Conversation

@PhilWakelin
Copy link
Copy Markdown
Member

@PhilWakelin PhilWakelin commented Mar 31, 2026

  • Simplified choices for deployment
  • Clarified terminology and Eclipse menus
  • Merged Eclipse CICS bundle project with Maven/Gradle bundle project
  • Removed Maven/Gradle nature from CICS bundle project

@PhilWakelin PhilWakelin changed the title README clarifications Merge Eclipse projects and clarify README Mar 31, 2026
Comment thread README.md

- Clone the repository using your IDEs support, such as the Eclipse Git plugin
- **or**, download the sample as a [ZIP](https://github.com/cicsdev/cics-java-liberty-restapp/archive/main.zip) and unzip onto the workstation
- Download the sample as a [ZIP](archive/refs/heads/main.zip) and unzip onto the workstation or clone the repository using a git client.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think the emphasis is the wrong way round. Cloning in Eclipse is by far the least error-prone approach.

Comment thread README.md
> In Eclipse, Gradle (buildship) is able to fully refresh and resolve the local classpath even if the project was previously updated by Maven. However, Maven (m2e) does not currently reciprocate that capability. If you previously refreshed the project with Gradle, you'll need to manually remove the 'Project Dependencies' entry on the Java build-path of your Project Properties to avoid duplication errors when performing a Maven Project Update.
Choose from the following 2 main approaches:

1. Use the command line to drive the supplied Gradle or Apache Maven Wrappers, this means there is no requirement for Gradle, Maven, Eclipse, or CICS Explorer SDK to be installed.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I'm not sure about this section. I think we are losing important build information and options.

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.

3 participants