search

microsoft / OpenAPI.NET.OData

Generates OpenAPI document from OData CSDL
MIT License
204 stars 62 forks source link

Bump Microsoft.OData.Edm from 7.21.3 to 8.0.0 #563

Closed dependabot[bot] closed 1 month ago

dependabot[bot] commented 1 month ago

Bumps Microsoft.OData.Edm from 7.21.3 to 8.0.0.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually 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 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)
andrueastman commented 1 month ago

@millicentachieng @irvinesunday Any chance you can confirm if the test failure is expected in the major version bump of the dependency?

baywet commented 1 month ago

@andrueastman and others on the thread. I think we should major rev with this bump since the dependency has lots of breaking changes and consumers would be "forced" to upgrade if we merged that. We could take that opportunity to clean up all the things marked as obsolete as well.

baywet commented 1 month ago

@andrueastman and others on the thread. I think we should major rev with this bump since the dependency has lots of breaking changes and consumers would be "forced" to upgrade if we merged that. We could take that opportunity to clean up all the things marked as obsolete as well.

@andrueastman @millicentachieng @irvinesunday actually I just made those changes + a couple of other cleanups. Let me know what you think!

sonarcloud[bot] commented 1 month ago

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint