X Tutup
The Wayback Machine - https://web.archive.org/web/20220323140421/https://github.com/nodejs/node/pull/40017
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

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

Closed
wants to merge 1 commit into from

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Sep 6, 2021

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

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

@cclauss cclauss added the python label Sep 6, 2021
@nodejs-github-bot nodejs-github-bot added build needs-ci labels Sep 6, 2021
@cclauss cclauss requested review from richardlau and targos Sep 6, 2021
targos
targos approved these changes Sep 6, 2021
Trott
Trott approved these changes Sep 7, 2021
jasnell
jasnell approved these changes Sep 7, 2021
Leko
Leko approved these changes Sep 7, 2021
@richardlau richardlau added the request-ci label Sep 7, 2021
@github-actions github-actions bot removed the request-ci label Sep 7, 2021
@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Sep 7, 2021

@BridgeAR BridgeAR added fast-track and removed fast-track labels Sep 7, 2021
@github-actions
Copy link

@github-actions github-actions bot commented Sep 7, 2021

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

@nodejs nodejs deleted a comment from github-actions bot Sep 7, 2021
@targos targos added the request-ci label Sep 8, 2021
@github-actions github-actions bot removed the request-ci label Sep 8, 2021
@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Sep 8, 2021

@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Sep 23, 2021

targos added a commit that referenced this issue 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
Member

@targos targos commented Sep 26, 2021

Landed in b25a73f

@targos targos closed this Sep 26, 2021
@targos targos deleted the configure.py-str.endswith-takes-a-tuple branch Sep 26, 2021
targos added a commit that referenced this issue 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 free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build fast-track needs-ci python
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

10 participants
X Tutup