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

build(deps): bump sphinx-tabs from 3.3.1 to 3.4.1 #20948

Closed
wants to merge 1 commit into from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Aug 17, 2022

Bumps sphinx-tabs from 3.3.1 to 3.4.1.

Release notes

Sourced from sphinx-tabs's releases.

Version 3.4.1

What's Changed

Full Changelog: executablebooks/sphinx-tabs@v3.4.0...v3.4.1

Version 3.4.0

What's Changed

New Contributors

Full Changelog: executablebooks/sphinx-tabs@v3.3.1...v3.4.0

Changelog

Sourced from sphinx-tabs's changelog.

3.4.1 - 2022-97-02

Added

  • Weekly scheduled testing, to catch breaking changes in unpinned dependencies

Changed

  • docutils version pin to allow use of verison 0.18.x

Removed

  • sphinx version pinning - only the latest version of sphinx will now be fully supported, but previous versions will work if sphinx dependencies (i.e. jinja2) are managed correctly. This is inline with the approach at sphinx
  • tests that were specific to older versions of sphinx and pygments
  • jinja2 version pinning, as this is now pinned in latest version of sphinx

3.4.0 - 2022-06-26

Added

  • Testing for sphinx 5
  • Tesing for python 3.10

Fixed

  • Fixed parsing of MyST content, where first line was being stripped
  • Typos in documentation
  • Failing regression tests

Changed

  • Testing to use an up-to-date pytest version

Removed

  • Testing for python 3.6 and sphinx versions 2 and 4 (see #164). Note that the package will likely continue to work fine with these, but this won't be assured by tests
Commits

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 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 requested a review from a team as a code owner August 17, 2022 15:01
@dependabot dependabot bot added kind/enhancement This would improve or streamline existing functionality. release-note/misc This PR makes changes that have no direct user impact. labels Aug 17, 2022
@dependabot dependabot bot requested a review from qmonnet August 17, 2022 15:01
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Aug 17, 2022
@tklauser
Copy link
Member

@dependabot rebase

@dependabot dependabot bot force-pushed the dependabot/pip/sphinx-tabs-3.4.1 branch from ad1e752 to 7a72bbc Compare August 17, 2022 15:21
@qmonnet
Copy link
Member

qmonnet commented Aug 18, 2022

This is blocked by #20950

@qmonnet qmonnet added the dont-merge/blocked Another PR must be merged before this one. label Aug 18, 2022
Copy link
Member

@qmonnet qmonnet left a 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)

@dependabot dependabot bot force-pushed the dependabot/pip/sphinx-tabs-3.4.1 branch from 7a72bbc to 328f6b9 Compare August 19, 2022 14:43
@qmonnet
Copy link
Member

qmonnet commented Aug 19, 2022

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.

qmonnet added a commit to qmonnet/cilium that referenced this pull request Aug 19, 2022
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>
joestringer pushed a commit that referenced this pull request Aug 19, 2022
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>
@qmonnet
Copy link
Member

qmonnet commented Aug 19, 2022

https://github.com/dependabot rebase

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>
@dependabot dependabot bot force-pushed the dependabot/pip/sphinx-tabs-3.4.1 branch from 328f6b9 to 0bac897 Compare August 19, 2022 21:14
@qmonnet
Copy link
Member

qmonnet commented Aug 19, 2022

Nice, it updates requirements-min.txt as well! Still blocked because of docutils though.

@qmonnet
Copy link
Member

qmonnet commented Aug 24, 2022

Closing given #20948 (comment)
@dependabot close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dont-merge/blocked Another PR must be merged before this one. kind/enhancement This would improve or streamline existing functionality. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants