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
allow optional types for string and map #601
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.
Good, thanks!
Now the bookkeeping questions:
- will we add a RPM dependency because of the new defines?
- will we need a bsc/jsc for the changelog?
@@ -2,28 +2,36 @@ default namespace = "http://www.suse.com/1.0/yast2ns" | |||
namespace a = "http://relaxng.org/ns/compatibility/annotations/1.0" | |||
namespace config = "http://www.suse.com/1.0/configns" | |||
|
|||
STRING_BOOL = STRING_ATTR, ("true" | "false") |
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.
we should still
include "common.rnc"
even if we don't run the validation yet
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.
how it will behave in final build which include everything? will be common.rng included many times?
@mvidner - to your question, no dependency, as modules should not depend on autoyast ( you do not want it on running system ). Also new references is not needed for trang, so only in yast2-schema it is good to have it together. for BSC I created https://bugzilla.suse.com/show_bug.cgi?id=1170886 |
* master: Bump version Update changes file Set AutoInstClonable in the desktop file
✔️ Public Jenkins job #54 successfully finished |
trello: https://trello.com/c/HkFkUQHj/1791-3-continue-with-new-xml-parser-xml-validation
depends on yast/yast-autoinstallation#598
Agreed to postpone for now that trang and jing travis validation to not delay even more new xml parser.