Skip to content

Python str.endswith() can take a tuple of alternatives - #40017

Closed
cclauss wants to merge 1 commit into
masterfrom
configure.py-str.endswith-takes-a-tuple
Closed

Python str.endswith() can take a tuple of alternatives#40017
cclauss wants to merge 1 commit into
masterfrom
configure.py-str.endswith-takes-a-tuple

Conversation

@cclauss

Copy link
Copy Markdown
Contributor

https://docs.python.org/3/library/stdtypes.html#str.endswith

One function call should be faster than three function calls per file.

@cclausscclauss added the python PRs and issues that require attention from people who are familiar with Python. label Sep 6, 2021
@nodejs-github-botnodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels Sep 6, 2021
Leko
Leko approved these changes Sep 7, 2021
@richardlaurichardlau added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 7, 2021
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 7, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@BridgeARBridgeAR added fast-track PRs that do not need to wait for 48 hours to land. and removed fast-track PRs that do not need to wait for 48 hours to land. labels Sep 7, 2021
@github-actions

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @BridgeAR. Please 👍 to approve.

@nodejsnodejs deleted a comment from github-actionsBotSep 7, 2021
@targostargos added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 8, 2021
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 8, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

targos pushed a commit that referenced this pull request Sep 26, 2021
https://docs.python.org/3/library/stdtypes.html#str.endswith
One function call should be faster than three function calls per file.
PR-URL: #40017
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
@targos

Copy link
Copy Markdown
Member

Landed in b25a73f

@targostargos closed this Sep 26, 2021
@targos
targos deleted the configure.py-str.endswith-takes-a-tuple branch September 26, 2021 09:00
targos pushed a commit that referenced this pull request Oct 4, 2021
https://docs.python.org/3/library/stdtypes.html#str.endswith
One function call should be faster than three function calls per file.
PR-URL: #40017
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildIssues and PRs related to build files or the CI.fast-trackPRs that do not need to wait for 48 hours to land.needs-ciPRs that need a full CI run.pythonPRs and issues that require attention from people who are familiar with Python.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@cclauss@nodejs-github-bot@targos@jasnell@Trott@Leko@tniessen@richardlau@RaisinTen@BridgeAR