-
Notifications
You must be signed in to change notification settings - Fork 22
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
Remove problematic ciriquant-yml process #113
Conversation
|
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.
LGTM, any reason why it's not an official module though?
Also may want a changelog update (unless not usually done for such changes in this project)
As this pipeline still has not had its 1.0 release, we haven't cared too much about a changelog yet. About official modules: There are several modules in this pipeline that could qualify to be added to the official modules, I am currently getting into module contributions and there will be PRs soonish. Thanks for the review! |
The process in question creates a YML file with absolute paths to working directories of other. I think this could have problematic effects with caching. Also, I made a change that makes the process more robust to problems as described in #109.