-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: E2E example with static files required to execute it #46
Conversation
…d functions used in templates
… the issues from `golangci-lint run --enable-all`
…hich from now is used also in config.tmpl
# Conflicts: # pkg/translate/funcs.go
491b5e3
to
ec3c02e
Compare
In folder |
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.
I've updated some of these files since this PR has been opened. Looking at assets/errors/panos.go
I don't see the UUID error I added as part of the UUID-style-Entry support. Just grab the new stuff.
I updated |
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.
GJ! 👍
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.
So pango internals are not "fully baked" yet, there is still a lot that needs to be added in... But this is fine as a starting point for the generator.
Description
PR delivers:
In folder
assets
all folders (besidesexample
) are copied fromdevelop
of repository https://github.com/PaloAltoNetworks/pango. Issue #49 was opened to refactor folderassets
.Motivation and Context
PR was opened to easily check , if autogenerated code works.
How Has This Been Tested?
Code was tested by executing command from README:
Types of changes
Checklist