Skip to content

DBB zBuilder samples and extensions folder#357

Merged
dennis-behm merged 3 commits into
IBM:mainfrom
drbruce-git:zBuilderExtensions2
Sep 11, 2025
Merged

DBB zBuilder samples and extensions folder#357
dennis-behm merged 3 commits into
IBM:mainfrom
drbruce-git:zBuilderExtensions2

Conversation

@drbruce-git
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Member

@dennis-behm dennis-behm left a comment

Choose a reason for hiding this comment

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

Hi @drbruce-git ,

I have added a view comments. Can you please take a look?

In addition, I am wondering if we should should plan for a set of subfolders under zBuilder:

  • Extensions (like in this PR)
  • Language Tasks (new samples, for instance the Multi Runtime sample)
  • sample/MortgageApplication

Thanks.

Comment thread zBuilder/MortgageApplication/README.md Outdated
Comment thread zBuilder/MortgageApplication/copybook/epsmortf.cpy Outdated
Comment thread zBuilder/MortgageApplication/copybook/epsnbrpm.cpy Outdated
Comment thread zBuilder/CheckFileExistsVariable/README.md Outdated
Comment thread zBuilder/CheckFileExistsVariable/groovy/CheckFileExists.groovy Outdated
Comment thread zBuilder/LanguageConfiguration/README.md Outdated
Comment thread zBuilder/LanguageConfiguration/README.md Outdated
Comment thread zBuilder/LanguageConfiguration/README.md Outdated
Comment thread zBuilder/LanguageConfiguration/README.md
Comment thread zBuilder/LanguageConfiguration/README.md Outdated
@drbruce-git
Copy link
Copy Markdown
Collaborator Author

Hi @dennis-behm I think I addressed all your suggestions. I don't want to create sub-folders at this time since MortgageApplication is already pretty deep. In the future if we start getting a lot of entries (say 10+) we can reconsider adding sub-folders.

Copy link
Copy Markdown
Member

@dennis-behm dennis-behm left a comment

Choose a reason for hiding this comment

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

Thank you @drbruce-git

@dennis-behm dennis-behm merged commit 0f8a61f into IBM:main Sep 11, 2025
1 check failed
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