-
Notifications
You must be signed in to change notification settings - Fork 0
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
Hotfix canary patch #19
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Akshay Mestry <[email protected]>
Signed-off-by: Akshay Mestry <[email protected]>
Signed-off-by: Akshay Mestry <[email protected]>
Signed-off-by: Akshay Mestry <[email protected]>
Impact Analysis: - For some reason, the JupyterLite patch failed on the CI checkout. This seems quite strange as on the all the local and SI testing, it did seem to work. But nonetheless, removed it for now to test the overall deployment process. Signed-off-by: Akshay Mestry <[email protected]>
Signed-off-by: Akshay Mestry <[email protected]>
Impact Analysis: - In all the internal testings, everything seemed to be quite right yet the JupyterLite seemed to cause some issues which are currently not yet identified to be resolved for a complete proof. - Hence, removing the inputs where JupyterLite was considered as important. This is by no means trying to put an end to the use of JupyterLite, we very much will investigate this further down the road but as of now, it seems not relevant. Signed-off-by: Akshay Mestry <[email protected]>
Signed-off-by: Akshay Mestry <[email protected]>
Signed-off-by: Akshay Mestry <[email protected]>
xames3
added
duplicate
this issue or pull request already exists
invalid
not a bug but, this pull request or issue doesn't seem right
wontfix
this pull request or issue will not be worked on
approved
pull requests that have passed review and are cleared for merging
ci/cd
modifications to CI/CD pipelines or processes
work in progress
pull requests that are not yet ready for review or merging
labels
Oct 31, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
approved
pull requests that have passed review and are cleared for merging
ci/cd
modifications to CI/CD pipelines or processes
duplicate
this issue or pull request already exists
invalid
not a bug but, this pull request or issue doesn't seem right
wontfix
this pull request or issue will not be worked on
work in progress
pull requests that are not yet ready for review or merging
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.