Skip to content

gh-153934: Correct documented param from n to width#153942

Merged
ZeroIntensity merged 1 commit into
python:mainfrom
podmar:fix-documented-weekheader-param
Jul 18, 2026
Merged

gh-153934: Correct documented param from n to width#153942
ZeroIntensity merged 1 commit into
python:mainfrom
podmar:fix-documented-weekheader-param

Conversation

@podmar

@podmar podmar commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Documentation

Documentation contains the following signature:

.. function:: weekheader(n)

   Return a header containing abbreviated weekday names. *n* specifies the width in
   characters for one weekday.

In reality the function uses "width" as the parameter name:

def formatweekheader(self, width):
        """
        Return a header for a week.
        """
        header = super().formatweekheader(width)
        theme = self._get_theme().calendar
        return f"{theme.weekday}{header}{theme.reset}"

Documentation in the calendar.rst file should be updated to reflect the actual implementation.

@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #33645969 | 📁 Comparing b1db24d against main (0e81ef9)

  🔍 Preview build  

1 file changed
± library/calendar.html

@github-project-automation github-project-automation Bot moved this from Todo to In Progress in Sprint Jul 18, 2026
@ZeroIntensity ZeroIntensity added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 pre-release feature fixes, bugs and security fixes labels Jul 18, 2026
@ZeroIntensity
ZeroIntensity merged commit 7486c7b into python:main Jul 18, 2026
48 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Docs PRs Jul 18, 2026
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Sprint Jul 18, 2026
@miss-islington-app

Copy link
Copy Markdown

Thanks @podmar for the PR, and @ZeroIntensity for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14, 3.15.
🐍🍒⛏🤖

@tomasr8

tomasr8 commented Jul 18, 2026

Copy link
Copy Markdown
Member

Thank you @podmar !!

@bedevere-app

bedevere-app Bot commented Jul 18, 2026

Copy link
Copy Markdown

GH-153943 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Jul 18, 2026
@bedevere-app

bedevere-app Bot commented Jul 18, 2026

Copy link
Copy Markdown

GH-153944 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Jul 18, 2026
@bedevere-app

bedevere-app Bot commented Jul 18, 2026

Copy link
Copy Markdown

GH-153945 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Jul 18, 2026
ZeroIntensity pushed a commit that referenced this pull request Jul 18, 2026
…r` from `n` to `width` (GH-153942) (GH-153944)

(cherry picked from commit 7486c7b)

Co-authored-by: Marta <marta.podziewska@gmail.com>
ZeroIntensity pushed a commit that referenced this pull request Jul 18, 2026
…r` from `n` to `width` (GH-153942) (GH-153945)

(cherry picked from commit 7486c7b)

Co-authored-by: Marta <marta.podziewska@gmail.com>
ZeroIntensity pushed a commit that referenced this pull request Jul 18, 2026
…r` from `n` to `width` (GH-153942) (GH-153943)

(cherry picked from commit 7486c7b)

Co-authored-by: Marta <marta.podziewska@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip news sprint

Projects

Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants