Skip to content

CSPL-4594 Fix optional fields to use omitempty#1745

Open
kasiakoziol wants to merge 4 commits intodevelopfrom
feature/CSPL-4594-fix-optional-fields-in-spec
Open

CSPL-4594 Fix optional fields to use omitempty#1745
kasiakoziol wants to merge 4 commits intodevelopfrom
feature/CSPL-4594-fix-optional-fields-in-spec

Conversation

@kasiakoziol
Copy link
Collaborator

@kasiakoziol kasiakoziol commented Mar 3, 2026

Description

It fixes issues with optional fields that do not pass the validation.

Key Changes

  • All optional fields get omitempty

Testing and Verification

Automated tests

Related Issues

Jira: https://splunk.atlassian.net/browse/CSPL-4594

PR Checklist

  • Code changes adhere to the project's coding standards.
  • Relevant unit and integration tests are included.
  • Documentation has been updated accordingly.
  • All tests pass locally.
  • The PR description follows the project's guidelines.

@coveralls
Copy link
Collaborator

coveralls commented Mar 3, 2026

Pull Request Test Coverage Report for Build 22706607596

Details

  • 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 83.937%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/splunk/enterprise/util.go 0 1 0.0%
Totals Coverage Status
Change from base Build 22574097150: 0.0%
Covered Lines: 11836
Relevant Lines: 14101

💛 - Coveralls

@kasiakoziol kasiakoziol force-pushed the feature/CSPL-4594-fix-optional-fields-in-spec branch from e3d6f43 to b046804 Compare March 3, 2026 11:38
@kasiakoziol kasiakoziol force-pushed the feature/CSPL-4594-fix-optional-fields-in-spec branch from b536acd to b40bcb4 Compare March 4, 2026 13:10
@kasiakoziol kasiakoziol force-pushed the feature/CSPL-4594-fix-optional-fields-in-spec branch from 9ff9e15 to 30177ea Compare March 4, 2026 17:53
@kasiakoziol kasiakoziol force-pushed the feature/CSPL-4594-fix-optional-fields-in-spec branch from 30177ea to d4a9739 Compare March 5, 2026 07:10
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

Successfully merging this pull request may close these issues.

7 participants