Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Modify behaviour so that failing to provide a JVM server when required generates an error #12

Closed
davenice opened this issue Jan 7, 2020 · 0 comments · Fixed by #49
Closed

Modify behaviour so that failing to provide a JVM server when required generates an error #12

davenice opened this issue Jan 7, 2020 · 0 comments · Fixed by #49
Assignees
@tom-foyle
Labels
MVP

Comments

@davenice
Copy link
Contributor

davenice commented Jan 7, 2020 •
edited by ledina
Loading

The behaviour should match the Maven plugin, issue IBM/cics-bundle-maven#138 - see discussion in there.

Implementation:
Reinstate code to throw error when JVM server not supplied, but only at the point where it is actually required. In other words, a bundle with no Java based parts will not give an error.
@ledina ledina added the MVP label Jan 16, 2020
@tom-foyle tom-foyle self-assigned this Feb 18, 2020
@ledina ledina closed this as completed Feb 26, 2020
@vera-chan vera-chan linked a pull request Feb 27, 2020 that will close this issue
update default JVM server name in samples & readme #49
Merged
sophiegreen pushed a commit to sophiegreen/cics-bundle-gradle that referenced this issue Jun 17, 2020
@banjaxxer
Convert first test back to Spock style and verify file copied
629a67a 
File copied to expected output folder
Issue IBM#12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tom-foyle tom-foyle

Labels
MVP
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

update default JVM server name in samples & readme vera-chan/cics-bundle-gradle
3 participants
@ledina @davenice @tom-foyle