Skip to content

Conversation

alichherawalla
Copy link

@alichherawalla alichherawalla commented Nov 10, 2021

Hi,

Thank you for this amazing library. It has truly made my life much simpler.

Currently, if I exclude an array element, it is replaced with null. For a certain use case my Cloudformation template breaks due to the null replacement. Hence if the item is an array I iterate over it and splice the index that isn't required

@alichherawalla alichherawalla force-pushed the master branch 3 times, most recently from b5d1f1d to e862289 Compare November 10, 2021 12:46
@alxAgu
Copy link

alxAgu commented Jan 11, 2022

Any chance of getting @alichherawalla 's commit merged? I'm having the exact same issue.

@alichherawalla
Copy link
Author

@anantab Would it be possible to please review this PR?

@alichherawalla
Copy link
Author

Thanks for the review @dapriett

Possible to please merge since I don't have write access

@alichherawalla
Copy link
Author

@anantab @dapriett possible to please merge this in?

@dapriett
Copy link

Hi @alichherawalla, sorry for the late reply. Unfortunately, I'm not one of the maintainers. Doesn't appear this repo is still being maintained.

But your change looked good to me. I was running into the same issue.

@kamilpp
Copy link

kamilpp commented Mar 23, 2023

Hi @anantab. Any chance we can get this or #42 get merged and published?

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.

4 participants