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

chore: make empty data cleanup more aggressive #82

Merged
merged 1 commit into from
Aug 28, 2024
Merged

chore: make empty data cleanup more aggressive #82

merged 1 commit into from
Aug 28, 2024

Conversation

mistahj67
Copy link
Contributor

Description

Part 1 of BED-4586

We currently do not strip empty top level structs or slices, this is a small change to make the stripping utility more aggressive.

Motivation and Context

Decrease file size by removing empty properties

How Has This Been Tested?

Ran locally
Updated unit tests

Types of changes

  • Chore (a change that does not modify the application functionality)
  • Could be considered breaking in the empty fields will no longer be present that were previously present

Checklist:

  • I have met the contributing prerequisites
    • Assigned myself to this PR
    • Added the appropriate labels
  • I have followed proper test practices
    • Added/updated tests to cover my changes
    • All new and existing tests passed

@sircodemane sircodemane changed the title chore: make stripper more aggressive chore: make empty data cleanup more aggressive Aug 28, 2024
@mistahj67 mistahj67 merged commit ed6bdf0 into main Aug 28, 2024
9 checks passed
@mistahj67 mistahj67 deleted the BED-4586 branch August 28, 2024 17:58
@github-actions github-actions bot locked and limited conversation to collaborators Aug 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants