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(deps): bump @xmldom/xmldom from 0.8.10 to 0.9.0; run [DynamicXml] #10481

Merged
merged 2 commits into from
Sep 7, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Aug 30, 2024

Bumps @xmldom/xmldom from 0.8.10 to 0.9.0.

Release notes

Sourced from @​xmldom/xmldom's releases.

0.9.0

Commits

Features

Fixed

Other

Thank you, @​kboshold, @​edi9999, @​apupier, @​shunkica, @​homer0, @​jhauga, @​UdayKharatmol, for your contributions

0.9.0-beta.11

Commits

Fixed

Other

Thank you, @​brodybits, @​cbettinger, @​josecarlosrx, for your contributions

0.9.0-beta.10

... (truncated)

Changelog

Sourced from @​xmldom/xmldom's changelog.

0.9.0

Features

Fixed

Other

Thank you, @​kboshold, @​edi9999, @​apupier, @​shunkica, @​homer0, @​jhauga, @​UdayKharatmol, for your contributions

0.9.0-beta.11

Fixed

Other

Thank you, @​brodybits, @​cbettinger, @​josecarlosrx, for your contributions

0.9.0-beta.10

Fixed

... (truncated)

Commits
  • 8c7c7af 0.9.0
  • 0e7afbb docs: prepare 0.9.0 (#709)
  • ade64e1 chore(deps-dev): bump axios from 1.6.0 to 1.7.5 (#707)
  • 2f598eb chore(deps): update github/codeql-action action to v3.26.5 (#706)
  • bccf286 chore(deps): update github/codeql-action action to v3.26.4 (#704)
  • 87fbb1f chore(deps): update actions/upload-artifact action to v4.3.6 (#703)
  • 3f14422 chore(deps): update dependency eslint-plugin-n to v17.10.2 (#700)
  • 94a0e11 chore(deps): update step-security/harden-runner action to v2.9.1 (#702)
  • 03e53b1 chore(deps): update actions/upload-artifact action to v4.3.5 (#701)
  • 4254b32 chore(deps): update dependency eslint-plugin-n to v17.10.0 (#699)
  • Additional commits viewable in compare view

Dependabot compatibility score

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added dependencies Related to dependency updates javascript Pull requests that update Javascript code labels Aug 30, 2024
Copy link

socket-security bot commented Aug 30, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@xmldom/[email protected] None 0 307 kB karfau

🚮 Removed packages: npm/@xmldom/[email protected])

View full report↗︎

@chris48s chris48s changed the title chore(deps): bump @xmldom/xmldom from 0.8.10 to 0.9.0 chore(deps): bump @xmldom/xmldom from 0.8.10 to 0.9.0; run [DynamicXml] Sep 2, 2024
@chris48s
Copy link
Member

chris48s commented Sep 2, 2024

@dependabot rebase

Bumps [@xmldom/xmldom](https://github.com/xmldom/xmldom) from 0.8.10 to 0.9.0.
- [Release notes](https://github.com/xmldom/xmldom/releases)
- [Changelog](https://github.com/xmldom/xmldom/blob/master/CHANGELOG.md)
- [Commits](xmldom/xmldom@0.8.10...0.9.0)

---
updated-dependencies:
- dependency-name: "@xmldom/xmldom"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/xmldom/xmldom-0.9.0 branch from ebc0166 to c4d7785 Compare September 2, 2024 13:29
Copy link
Contributor

github-actions bot commented Sep 2, 2024

Warnings
⚠️ This PR modified service code for dynamic but not its test code.
That's okay so long as it's refactoring existing code.
Messages
📖

✨ Thanks for your contribution to Shields, @dependabot[bot]!

Generated by 🚫 dangerJS against d212df5

const parsed = new DOMParser().parseFromString(buffer)
const parsed = new DOMParser().parseFromString(buffer, 'text/xml')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Notes on breaking change:
https://github.com/xmldom/xmldom/blob/master/CHANGELOG.md#fixed-27
In principle, this might be breaking if any users were using the dynamic xml badge to try and parse HTML documents (which may or may not have worked). One thing we could do here is try and decide whether we should pass text/html here based on the filename or response headers. I think my instinct on this is to just hard-code text/xml here, deploy it, and see if anyone pops up demanding we re-enable spacebar heating

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds reasonable to me.

@chris48s
Copy link
Member

chris48s commented Sep 2, 2024

I'm going to leave this one open for a bit before I merge it, just in case anyone wants to chime in on this.

Copy link
Contributor Author

dependabot bot commented on behalf of github Sep 6, 2024

A newer version of @​xmldom/xmldom exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged.

@chris48s
Copy link
Member

chris48s commented Sep 7, 2024

Right. I'm going to go ahead and merge this then let dependabot bump us to 0.9.2. It will probably be quicker than editing this

@chris48s chris48s added this pull request to the merge queue Sep 7, 2024
Merged via the queue into master with commit ffd0903 Sep 7, 2024
23 checks passed
@chris48s chris48s deleted the dependabot/npm_and_yarn/xmldom/xmldom-0.9.0 branch September 7, 2024 18:01
Copy link

sentry-io bot commented Sep 7, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ ParseError: Opening and ending tag mismatch: "link" != "head" DynamicXml.transform(services.dynamic:dynamic-x... View Issue

Did you find this useful? React with a 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Related to dependency updates javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants