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.

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

No documentation for format of --exclude and --include options #305

Open
werdnum opened this issue Jun 6, 2023 · 1 comment
Open

No documentation for format of --exclude and --include options #305

werdnum opened this issue Jun 6, 2023 · 1 comment

Comments

@werdnum
Copy link

werdnum commented Jun 6, 2023

It's very unclear what format the --exclude and --include options are supposed to take.

A bit of code-diving indicates that maybe they can take the "Go-Patch" style or the "Dot-Style" formats, but I have no idea what those are and I can't turn up any actual specifications for these formats – I can barely even find examples.

The --exclude-regexp and --include-regexp options are even more opaque, but reading through the code it seems like these regexps are maybe applied to "Go-Patch style" paths (per #266)?

I also note that even when I seemingly correctly specify --exclude-regexp '\/(status|metadata)\/.*', I still get the following in the output:

/
- one map entry removed:
  status:
    loadBalancer: {}

... presumably because the 'change' is the removal of a child of a root element rather than deletion of the 'status' element, which seems a particularly obtuse way of interpreting my intent.

Can we please have some documentation for how these fields are parsed and applied?

@tavlima
Copy link

tavlima commented Sep 19, 2023

This is as far as I could track it down. At a glance, still cryptic... 😩

https://github.com/gonvenience/ytbx/blob/main/path.go

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants