Skip to content

Conversation

sobolevn
Copy link
Member

@sobolevn sobolevn commented Apr 11, 2023

@sobolevn sobolevn added skip news build The build process and cross-build labels Apr 11, 2023
@sobolevn sobolevn changed the title gh-103180: Add CI timeouts to all GtiHub Actions jobs gh-103180: Add CI timeouts to all GitHub Actions jobs Apr 11, 2023
Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the analysis at #103180 (comment).

We could probably skip timeouts for the short ones: timeouts tend to occur in the more complex and therefore slower ones, but I'm also fine adding them just in case.

@Ismaelkadir

This comment was marked as spam.

@sobolevn
Copy link
Member Author

@Ismaelkadir sorry, I don't understand your message :(

@Ismaelkadir

This comment was marked as spam.

Copy link
Contributor

@hauntsaninja hauntsaninja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this!

@hugovk hugovk merged commit be8903e into python:main Apr 14, 2023
@hugovk hugovk added the needs backport to 3.11 only security fixes label Apr 14, 2023
@miss-islington
Copy link
Contributor

Thanks @sobolevn for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@miss-islington
Copy link
Contributor

Sorry, @sobolevn and @hugovk, I could not cleanly backport this to 3.11 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker be8903eb9d66ef1229f93a3a6036aeafc3bb0bda 3.11

@sobolevn
Copy link
Member Author

Do you want me to backport this to 3.11 as well?

@hugovk
Copy link
Member

hugovk commented Apr 14, 2023

Yes please, could be useful there too :)

carljm added a commit to carljm/cpython that referenced this pull request Apr 14, 2023
* main:
  pythongh-103532: Remove TKINTER_PROTECT_LOADTK code (pythonGH-103535)
  pythongh-103180: Add CI timeouts to all GitHub Actions jobs (python#103437)
  Remove double space in import error message (python#103458)
  ipaddress: Remove non-existent ip_str param from docstring (python#103461)
  Fix syntax typo in isolating extensions doc (python#103516)
  pythongh-103406: Modernize pos-only arguments usage in `test_signature` (python#103407)
  Proofread howto/perf_profiling.rst (python#103530)
  Fix unused functions warnings in instrumentation.c (pythonGH-103515)
sobolevn added a commit to sobolevn/cpython that referenced this pull request Apr 14, 2023
…ythonGH-103437).

(cherry picked from commit be8903e)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
@bedevere-bot
Copy link

GH-103543 is a backport of this pull request to the 3.11 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Apr 14, 2023
carljm added a commit to carljm/cpython that referenced this pull request Apr 14, 2023
* superopt:
  fix incompatible types
  update generated cases
  don't unnecessarily re-find args in error case
  Apply suggestions from code review
  pythongh-103532: Remove TKINTER_PROTECT_LOADTK code (pythonGH-103535)
  pythongh-103180: Add CI timeouts to all GitHub Actions jobs (python#103437)
  Remove double space in import error message (python#103458)
  ipaddress: Remove non-existent ip_str param from docstring (python#103461)
  Fix syntax typo in isolating extensions doc (python#103516)
  pythongh-103406: Modernize pos-only arguments usage in `test_signature` (python#103407)
  Proofread howto/perf_profiling.rst (python#103530)
  Fix unused functions warnings in instrumentation.c (pythonGH-103515)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build The build process and cross-build skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants