Resolve PR feedback

Author: heaths

sdk/core/azure_core/README.md

@@ -180,7 +180,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
 
 ### Consuming service methods returning `Pager<T>`
 
-If a service call returns multiple values in pages, it would return `Result<Pager<T>>` as a result. You can iterate all items from all pages.
+If a service call returns multiple values in pages, it should return `Result<Pager<T>>` as a result. You can iterate all items from all pages.
 
 ```rust no_run
 use azure_identity::DefaultAzureCredential;
@@ -248,7 +248,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
 
 ### Consuming service methods returning `Poller<T>`
 
-If a service call may take a while to process, it would return `Result<Poller<T>>` as a result, representing a long-running operation (LRO).
+If a service call may take a while to process, it should return `Result<Poller<T>>` as a result, representing a long-running operation (LRO).
 The `Poller<T>` implements `futures::Stream` so you can asynchronously iterate over each status monitor update:
 
 ```rust no_run

sdk/core/azure_core/src/http/poller.rs

@@ -23,7 +23,7 @@ use std::{
 /// Default retry time for long-running operations if no retry-after header is present
 ///
 /// This value is the same as the default used in other Azure SDKs e.g.,
-/// <https://github.com/Azure/azure-sdk-for-python/blob/azure-mgmt-core_1.5.0/sdk/core/azure-mgmt-core/azure/mgmt/core/polling/arm_polling.py#L191>
+/// <https://github.com/Azure/azure-sdk-for-python/blob/azure-core_1.35.0/sdk/core/azure-core/azure/core/polling/base_polling.py#L586>
 const DEFAULT_RETRY_TIME: Duration = Duration::seconds(30);
 const MIN_RETRY_TIME: Duration = Duration::seconds(1);