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

Hotfix escape paths for ADLS #30

Merged
merged 10 commits into from
Feb 22, 2024
Merged

Conversation

ddl-gabrielhaim
Copy link
Collaborator

https://dominodatalab.atlassian.net/browse/DOM-54296

  • Pin reticulate version to 1.34.0
  • Add path escape hotfix
  • Fix parse -> compose
  • typo

R/http.R Outdated
#' @details Internal only to enable retry for malformed URLs.
#' @param verb A character string containing an HTTP verb
#' @param url Signed URL to download object at.
#' @param datasource_type Type of datasource the object is stored in.
Copy link

@ddl-joyce-zhao ddl-joyce-zhao Feb 22, 2024

Choose a reason for hiding this comment

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

nit: unused parameter datasource_type

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

thanks will cleanup

DESCRIPTION Outdated
@@ -1,6 +1,6 @@
Package: DominoDataR
Title: 'Domino Data R SDK'
Version: 0.2.3
Version: 0.2.5

Choose a reason for hiding this comment

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

Why skip 0.2.4?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I didn't skip it, it was released and I forgot to update.. Wondering if I should jump to 0.2.5 or keep 0.2.4 and re-release on github.

Copy link

@ddl-joyce-zhao ddl-joyce-zhao left a comment

Choose a reason for hiding this comment

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

LGTM

@ddl-gabrielhaim ddl-gabrielhaim merged commit c154e68 into main Feb 22, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants