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

fix(duplicated-defaults-keys): Fix Duplicated Default Keys Assigned via Object.assigbn #217

Merged
merged 4 commits into from
Feb 8, 2021

Conversation

EntraptaJ
Copy link
Member

I was attempting to use Winston and discovered a minor error, it's caused by modules using Object.assign to assign keys, which isn't picked up by the current filter

@EntraptaJ
Copy link
Member Author

This PR does have the possibility of serious breakage, I am unsure if any current usage of this module actually relies upon this "bug" so I'm going to merge in such a way that this will force a 2.0 release, need to figure out the right commit type with Semantic release to make this happen.

@EntraptaJ EntraptaJ enabled auto-merge (squash) February 8, 2021 19:26
@EntraptaJ EntraptaJ merged commit 5d819ce into master Feb 8, 2021
@EntraptaJ EntraptaJ deleted the fix-dupicated-defaults-keys branch February 8, 2021 19:27
KJDev-Bot added a commit that referenced this pull request Feb 8, 2021
# [2.0.0](v1.7.0...v2.0.0) (2021-02-08)

### Bug Fixes

* **Actions:** Fix Test action on Release Workflow ([082e58b](082e58b))
* **duplicated-defaults-keys:** Fix Duplicated Default Keys Assigned via Object.assigbn ([#217](#217)) ([5d819ce](5d819ce))

### BREAKING CHANGES

* **duplicated-defaults-keys:** This is potentially a major change, I'm uncertain if any modules or current projects actually rely upon this "bug" Please create an issue if anything is broken after this release.
@KJDev-Bot
Copy link
Contributor

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants