Skip to content

bpo-30444: Added ability to set pager module's see more text#1774

Closed
gautamkrishnar wants to merge 1 commit intopython:masterfrom
gautamkrishnar:pydoc_pager_fix
Closed

bpo-30444: Added ability to set pager module's see more text#1774
gautamkrishnar wants to merge 1 commit intopython:masterfrom
gautamkrishnar:pydoc_pager_fix

Conversation

@gautamkrishnar
Copy link
Copy Markdown

@gautamkrishnar gautamkrishnar commented May 23, 2017

As of now, their is no way to change the text -- more -- that is displayed while using pager module. This PR will introduce this feature to the pager module so that we can use pager module, more effectively in our own applications for piping the text.

Old usage

pager("text to pipe")

New usage

pager("text to pipe", "text to display in the see more section")

I am new to python project. Please guide me if i need to make any changes regarding this PR.

Issue: https://bugs.python.org/issue30444

@the-knights-who-say-ni
Copy link
Copy Markdown

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA).

Unfortunately we couldn't find an account corresponding to your GitHub username on bugs.python.org (b.p.o) to verify you have signed the CLA (this might be simply due to a missing "GitHub Name" entry in your b.p.o account settings). This is necessary for legal reasons before we can look at your contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

Thanks again to your contribution and we look forward to looking at it!

@mention-bot
Copy link
Copy Markdown

@gautamkrishnar, thanks for your PR! By analyzing the history of the files in this pull request, we identified @zestyping, @tim-one and @birkenfeld to be potential reviewers.

@gautamkrishnar gautamkrishnar changed the title Added ability to set pager module's see more text bpo-30444: Added ability to set pager module's see more text May 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants