-
Notifications
You must be signed in to change notification settings - Fork 13
feat: generate more accurate agent object #70
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
Conversation
Signed-off-by: András Jáky <[email protected]>
Signed-off-by: András Jáky <[email protected]>
Signed-off-by: András Jáky <[email protected]>
looking good, can we pass algo-type as part of the digest, we can use fixed algo for now for simplification. something like |
can we also strip the |
notes above may not be relevant for the current PR, but we do need to address these |
I'm addressing the extension part in the next PR a bit, but some additional changes will still need to be implemented serverside for perfect extension example generation. |
Signed-off-by: András Jáky <[email protected]>
Signed-off-by: András Jáky <[email protected]>
93384d5
to
b5072a6
Compare
* fix: add image back to dictionary * feat: add prefix to generated hashes Signed-off-by: András Jáky <[email protected]> * feat: add observability formats class as extension Signed-off-by: András Jáky <[email protected]> * feat: generate array of string enums Signed-off-by: András Jáky <[email protected]> * feat: only update class/category uid if is present Signed-off-by: András Jáky <[email protected]> * feat: modify base_feature So the extension schema has name and version but does not have unnecessary class and category names and uids. Signed-off-by: András Jáky <[email protected]> * feat: agent extension to point to feature classes Signed-off-by: András Jáky <[email protected]> --------- Signed-off-by: András Jáky <[email protected]>
Simplified digest and url attributes to generate better Agent jsons.