Skip to content

Commit

Permalink
Clean up some typos from translation
Browse files Browse the repository at this point in the history
  • Loading branch information
zsusswein committed Dec 6, 2024
1 parent 8994748 commit eacd5c3
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 8 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
ifndef TAG
TAG = latest
TAG = edit-azure-flow
endif

REGISTRY=cfaprdbatchcr.azurecr.io/
Expand Down
13 changes: 6 additions & 7 deletions R/azure.R
Original file line number Diff line number Diff line change
Expand Up @@ -109,10 +109,9 @@ fetch_blob_container <- function(container_name) {
)
cli::cli_alert_info("Loading Azure credentials from env vars")
# nolint start: object_name_linter
az_tenant_id <- fetch_credential_from_env_var("az_tenant_id ")
az_subscription_id <- fetch_credential_from_env_var("az_subscription_id")
az_resource_group <- fetch_credential_from_env_var("az_resource_group")
az_storage_account <- fetch_credential_from_env_var("az_storage_account")
az_tenant_id <- fetch_credential_from_env_var("az_tenant_id")
az_client_id <- fetch_credential_from_env_var("az_client_id")
az_service_principal <- fetch_credential_from_env_var("az_service_principal")

Check warning on line 114 in R/azure.R

View check run for this annotation

Codecov / codecov/patch

R/azure.R#L113-L114

Added lines #L113 - L114 were not covered by tests
# nolint end: object_name_linter
cli::cli_alert_success("Credentials loaded successfully")

Expand All @@ -122,9 +121,9 @@ fetch_blob_container <- function(container_name) {
{
token <- AzureRMR::get_azure_token(
resource = "https://storage.azure.com",
tenant = Sys.getenv("az_tenant_id"),
app = Sys.getenv("az_client_id"),
password = Sys.getenv("SERVICE_PRINCIPAL")
tenant = az_tenant_id,
app = az_client_id,
password = az_service_principal

Check warning on line 126 in R/azure.R

View check run for this annotation

Codecov / codecov/patch

R/azure.R#L122-L126

Added lines #L122 - L126 were not covered by tests
)
endpoint <- storage_endpoint(
"https://cfaazurebatchprd.blob.core.windows.net",
Expand Down

0 comments on commit eacd5c3

Please sign in to comment.