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

Fix genConfigSample gradle task #148

Closed
sebastian-peter opened this issue Feb 22, 2022 · 2 comments · Fixed by #131
Closed

Fix genConfigSample gradle task #148

sebastian-peter opened this issue Feb 22, 2022 · 2 comments · Fixed by #131
Assignees
@sebastian-peter
Milestone
Version 3.0

Comments

@sebastian-peter
Copy link
Member

sebastian-peter commented Feb 22, 2022 •
edited
Loading

Possible merge it with genConfigClass, since it requires parameter --dd for output anyway
@sebastian-peter sebastian-peter added this to the Version 3.0 milestone Feb 22, 2022
@sebastian-peter sebastian-peter self-assigned this Feb 22, 2022
@ckittl ckittl mentioned this issue Feb 22, 2022
Merged
@johanneshiry
Copy link
Member

Don't get the rationale behind this. Would you mind explaining this a little bit more detailed. AFAIK genConfigSample and genConfigClass do totally different things? Is this a code cleanup proposal or a vote for removing genConfigSample in favor of genConfigClass (which IMHO doesn't make sense).

@sebastian-peter
Copy link
Member Author

sebastian-peter commented Feb 23, 2022 •
edited
Loading

genConfigClass is broken in its current form, it throws a FileNotFoundException. This issue describes a possible solution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sebastian-peter sebastian-peter

Labels
None yet
Projects
None yet
Milestone
Version 3.0
Development

Successfully merging a pull request may close this issue.

Fix config auto-generation ie3-institute/simona
2 participants
@sebastian-peter @johanneshiry