Skip to content

Commit

Permalink
Merge #237
Browse files Browse the repository at this point in the history
237: Escape unused DateFormat characters (again) r=omus a=omus

Follow up to #236. I missed this use of `Dates.format`.

Co-authored-by: Curtis Vogt <[email protected]>
  • Loading branch information
bors[bot] and omus authored Feb 7, 2022
2 parents fc96007 + f9d5521 commit 844efdd
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions test/s3path.jl
Original file line number Diff line number Diff line change
Expand Up @@ -285,8 +285,9 @@ end

# This is the main entrypoint for the S3Path tests
function s3path_tests(config)
bucket_name =
"ocaws.jl.test." * lowercase(Dates.format(now(Dates.UTC), "yyyymmddTHHMMSSZ"))
bucket_name = let df = dateformat"yyyymmdd\THHMMSS\Z"
"ocaws.jl.test." * lowercase(Dates.format(now(Dates.UTC), df))
end

s3_create_bucket(config, bucket_name)
root = Path("s3://$bucket_name/pathset-root/")
Expand Down

0 comments on commit 844efdd

Please sign in to comment.