-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
build(deps): bump sphinx-tabs from 3.3.1 to 3.4.1 #20948
Conversation
@dependabot rebase |
ad1e752
to
7a72bbc
Compare
This is blocked by #20950 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Rejecting the current change as per my comment above)
7a72bbc
to
328f6b9
Compare
Looking further, we can't update to 3.4.1 because it requires docutils 0.18, which is not supported by the theme we use for the docs. However, we can update up to 3.4.0, which works with docutils 0.17. This should also allow us to bump sphinx to v5+. I'll submit a PR separately. |
We cannot bump sphinx-tabs to 3.4.1 as proposed by Dependabot in cilium#20948 because that version requires docutils 0.18, which is not supported yet by our theme (or the upstream version we derived it from). However, we can bump to 3.4.0 which does not have this requirement. This allows us to bump Sphinx to 5.1.1. Minor adjustments are required for the configuration file, to prevent warning when building the doc: We must set a language (I used "en" for English, the default fallback according to the warning), and we must have a single '%s' in the captions for extlinks (or use None for no caption). Signed-off-by: Quentin Monnet <quentin@isovalent.com>
We cannot bump sphinx-tabs to 3.4.1 as proposed by Dependabot in #20948 because that version requires docutils 0.18, which is not supported yet by our theme (or the upstream version we derived it from). However, we can bump to 3.4.0 which does not have this requirement. This allows us to bump Sphinx to 5.1.1. Minor adjustments are required for the configuration file, to prevent warning when building the doc: We must set a language (I used "en" for English, the default fallback according to the warning), and we must have a single '%s' in the captions for extlinks (or use None for no caption). Signed-off-by: Quentin Monnet <quentin@isovalent.com>
Bumps [sphinx-tabs](https://github.com/executablebooks/sphinx-tabs) from 3.3.1 to 3.4.1. - [Release notes](https://github.com/executablebooks/sphinx-tabs/releases) - [Changelog](https://github.com/executablebooks/sphinx-tabs/blob/master/CHANGELOG.md) - [Commits](executablebooks/sphinx-tabs@v3.3.1...v3.4.1) --- updated-dependencies: - dependency-name: sphinx-tabs dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
328f6b9
to
0bac897
Compare
Nice, it updates requirements-min.txt as well! Still blocked because of docutils though. |
Closing given #20948 (comment) |
Bumps sphinx-tabs from 3.3.1 to 3.4.1.
Release notes
Sourced from sphinx-tabs's releases.
Changelog
Sourced from sphinx-tabs's changelog.
Commits
9431223
RELEASE v3.4.126bad87
Change testing to only cover latest sphinx version (#167)b9fd7d8
RELEASE: v3.4.09b061d5
🐛 FIX: Fix to stop stripping of first line in MyST content (#165)f2b4714
👌 IMPROVE: Add support and testing for sphinx 5 (#164)88a096f
📚 DOCS: Fix typos (#157)c024ea8
[pre-commit.ci] pre-commit autoupdate (#156)193646f
Change git to https in pre-commit-hooks (#158)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 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)