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.

Sign up for GitHub

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

support JSON Merge Patch and Strategic Merge Patch in Resource Modifiers #6917

Merged
merged 13 commits into from
Nov 2, 2023

add more tests

6d89780
Select commit
Loading
Failed to load commit list.
Merged

support JSON Merge Patch and Strategic Merge Patch in Resource Modifiers #6917

add more tests
6d89780
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 24, 2023 in 1s

62.56% of diff hit (target 60.69%)

View this Pull Request on Codecov

62.56% of diff hit (target 60.69%)

Annotations

Check warning on line 25 in internal/resourcemodifiers/json_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/json_merge_patch.go#L24-L25 

Added lines #L24 - L25 were not covered by tests

Check warning on line 36 in internal/resourcemodifiers/json_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/json_merge_patch.go#L35-L36 

Added lines #L35 - L36 were not covered by tests

Check warning on line 43 in internal/resourcemodifiers/json_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/json_merge_patch.go#L42-L43 

Added lines #L42 - L43 were not covered by tests

Check warning on line 62 in internal/resourcemodifiers/json_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/json_patch.go#L62 

Added line #L62 was not covered by tests

Check warning on line 69 in internal/resourcemodifiers/json_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/json_patch.go#L68-L69 

Added lines #L68 - L69 were not covered by tests

Check warning on line 79 in internal/resourcemodifiers/json_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/json_patch.go#L78-L79 

Added lines #L78 - L79 were not covered by tests

Check warning on line 84 in internal/resourcemodifiers/json_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/json_patch.go#L83-L84 

Added lines #L83 - L84 were not covered by tests

Check warning on line 109 in internal/resourcemodifiers/resource_modifiers.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/resource_modifiers.go#L107-L109 

Added lines #L107 - L109 were not covered by tests

Check warning on line 171 in internal/resourcemodifiers/resource_modifiers.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/resource_modifiers.go#L171 

Added line #L171 was not covered by tests

Check warning on line 199 in internal/resourcemodifiers/resource_modifiers.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/resource_modifiers.go#L199 

Added line #L199 was not covered by tests

Check warning on line 204 in internal/resourcemodifiers/resource_modifiers.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/resource_modifiers.go#L204 

Added line #L204 was not covered by tests

Check warning on line 47 in internal/resourcemodifiers/strategic_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/strategic_merge_patch.go#L46-L47 

Added lines #L46 - L47 were not covered by tests

Check warning on line 69 in internal/resourcemodifiers/strategic_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/strategic_merge_patch.go#L68-L69 

Added lines #L68 - L69 were not covered by tests

Check warning on line 76 in internal/resourcemodifiers/strategic_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/strategic_merge_patch.go#L75-L76 

Added lines #L75 - L76 were not covered by tests

Check warning on line 80 in internal/resourcemodifiers/strategic_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/strategic_merge_patch.go#L79-L80 

Added lines #L79 - L80 were not covered by tests

Check warning on line 97 in internal/resourcemodifiers/strategic_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/strategic_merge_patch.go#L96-L97 

Added lines #L96 - L97 were not covered by tests

Check warning on line 111 in internal/resourcemodifiers/strategic_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/strategic_merge_patch.go#L102-L111 

Added lines #L102 - L111 were not covered by tests

Check warning on line 118 in internal/resourcemodifiers/strategic_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/strategic_merge_patch.go#L114-L118 

Added lines #L114 - L118 were not covered by tests

Check warning on line 121 in internal/resourcemodifiers/strategic_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/strategic_merge_patch.go#L121 

Added line #L121 was not covered by tests

Check warning on line 127 in internal/resourcemodifiers/strategic_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/strategic_merge_patch.go#L124-L127 

Added lines #L124 - L127 were not covered by tests

Check warning on line 142 in internal/resourcemodifiers/strategic_merge_patch.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/resourcemodifiers/strategic_merge_patch.go#L135-L142 

Added lines #L135 - L142 were not covered by tests

Check warning on line 1356 in pkg/restore/restore.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/restore/restore.go#L1356 

Added line #L1356 was not covered by tests
View more details on Codecov