Skip to content
This repository has been archived by the owner on Sep 2, 2021. It is now read-only.

adding maxWait command line flag #85

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

dharmaquark
Copy link

Description of change

This PR introduces a new optional command line flag max-wait to support customizable timeouts (implementing suggestion made in #80 to handle use case of an instance of JupyterLab taking longer than the current 5000 ms default before the window.jupyterlab object is available)

Bumps [tar](https://github.com/npm/node-tar) from 4.4.13 to 4.4.15.
- [Release notes](https://github.com/npm/node-tar/releases)
- [Changelog](https://github.com/npm/node-tar/blob/main/CHANGELOG.md)
- [Commits](isaacs/node-tar@v4.4.13...v4.4.15)

---
updated-dependencies:
- dependency-name: tar
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

adding maxWait command line flag

adding maxWait flag

adding maxWait flag
@welcome
Copy link

welcome bot commented Sep 1, 2021

Thanks for submitting your first pull request! You are awesome! 🤗

If you haven't done so already, check out Jupyter's Code of Conduct. Also, please make sure you followed the pull request template, as this will help us review your contribution more quickly.
welcome
You can meet the other Jovyans by joining our Discourse forum. There is also a intro thread there where you can stop by and say Hi! 👋

Welcome to the Jupyter community! 🎉

@jtpio
Copy link
Member

jtpio commented Sep 1, 2021

Thanks @dharmaquark!

Galata has been moved into the core JupyterLab repository: jupyterlab/jupyterlab#10796. And we are in the process of archiving the galata repo in: #81

However it looks like we could get this change in here, and cut a new @jupyterlab/galata release (maybe the last one from this repo). Then we can focus efforts on the main jupyterlab repo.

What do you think @fcollonval @mbektas?

@jtpio jtpio added the enhancement New feature or request label Sep 1, 2021
@dharmaquark
Copy link
Author

Thanks @dharmaquark!

Galata has been moved into the core JupyterLab repository: jupyterlab/jupyterlab#10796. And we are in the process of archiving the galata repo in: #81

However it looks like we could get this change in here, and cut a new @jupyterlab/galata release (maybe the last one from this repo). Then we can focus efforts on the main jupyterlab repo.

What do you think @fcollonval @mbektas?

that would be great! Also happy to port this PR over to the JupyterLab repo if that ends up being easier for everyone!

@fcollonval
Copy link
Member

FYI in the code ported to core, this is happening there. And there is no timeout on the action (except the global test timeout).

@mbektas
Copy link
Member

mbektas commented Sep 1, 2021

However it looks like we could get this change in here, and cut a new @jupyterlab/galata release (maybe the last one from this repo). Then we can focus efforts on the main jupyterlab repo.

What do you think @fcollonval @mbektas?

Thanks @dharmaquark! @jtpio yes, sounds good to me. Lets merge this and publish a new version.

@mbektas mbektas self-requested a review September 1, 2021 15:33
Copy link
Member

@mbektas mbektas left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@jtpio jtpio left a comment

Choose a reason for hiding this comment

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

Thanks!

@jtpio jtpio merged commit dcd7965 into jupyterlab:main Sep 1, 2021
@welcome
Copy link

welcome bot commented Sep 1, 2021

Congrats on your first merged pull request in this project! 🎉
congrats
Thank you for contributing, we are very proud of you! ❤️

@jtpio
Copy link
Member

jtpio commented Sep 2, 2021

FYI @dharmaquark, 3.0.11-4 has been published with this change: https://github.com/jupyterlab/galata/releases/tag/v3.0.11-4

Thanks @mbektas for the release!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants