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(developer): Update kmc-package xml2js dependency 🎺 #9716

Merged

Conversation

mcdurdin
Copy link
Member

@mcdurdin mcdurdin commented Oct 9, 2023

I missed adding this as a dependency for @keymanapp/kmc-package. because node has a very lenient dependency search, it found xml2js from other @keymanapp/common-types, so never caused errors locally.

@keymanapp-test-bot skip

I missed adding this as a dependency for @keymanapp/kmc-package. because
node has a very lenient dependency search, it found xml2js from other
@keymanapp/common-types, so never caused errors locally.
@mcdurdin mcdurdin requested a review from darcywong00 as a code owner October 9, 2023 00:45
@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Oct 9, 2023

User Test Results

Test specification and instructions

User tests are not required

Test Artifacts

@keymanapp-test-bot keymanapp-test-bot bot changed the title chore(developer): Update kmc-package xml2js dependency chore(developer): Update kmc-package xml2js dependency 🎺 Oct 9, 2023
@keymanapp-test-bot keymanapp-test-bot bot added this to the A17S23 milestone Oct 9, 2023
Base automatically changed from chore/common/9713-avoid-xml2js-firstCharLowerCase to epic/package-metadata October 11, 2023 00:19
@mcdurdin mcdurdin merged commit a4652ce into epic/package-metadata Oct 11, 2023
3 checks passed
@mcdurdin mcdurdin deleted the chore/update-kmp-compiler-xml2js-dependency branch October 11, 2023 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants