-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Spelling #1813
base: master
Are you sure you want to change the base?
Spelling #1813
Conversation
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
@@ -1 +1 @@ | |||
Please consult the documentation [here](https://github.com/Microsoft/cpprestsdk/wiki/How-to-build-for-iOS) for iOS compilation. | |||
Please consult the [documentation for iOS compilation](https://github.com/Microsoft/cpprestsdk/wiki/How-to-build-for-iOS). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is based on an accessibility complaint (for people using screen readers):
Do not use (click) here
links
For more information, see:
- https://www.w3.org/QA/Tips/noClickHere
- https://webaim.org/techniques/hypertext/link_text
- https://granicus.com/blog/why-click-here-links-are-bad/
- https://heyoka.medium.com/dont-use-click-here-f32f445d1021
(?i)(?:>|\[)(?:(?:click |)here|link|(?:read |)more)(?:</|\]\()
@@ -782,7 +782,7 @@ enum json_error | |||
malformed_numeric_literal, | |||
malformed_string_literal, | |||
malformed_token, | |||
mismatched_brances, | |||
mismatched_braces, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this could be an API break
@@ -17,7 +17,7 @@ | |||
<TranslateTransform/> | |||
</TransformGroup> | |||
</Canvas.RenderTransform> | |||
<Image HorizontalAlignment="Left" Source="Cards.PNG" Stretch="None" VerticalAlignment="Top" RenderTransformOrigin="0.05,0.08" x:Name="imgCard"> | |||
<Image HorizontalAlignment="Left" Source="cards.png" Stretch="None" VerticalAlignment="Top" RenderTransformOrigin="0.05,0.08" x:Name="imgCard"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the file is renamed below
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the corresponding reference is renamed above
@@ -1417,7 +1417,7 @@ utility::string_t __cdecl timespan::seconds_to_xml_duration(utility::seconds dur | |||
} | |||
|
|||
// The format is: | |||
// PdaysDThoursHminutesMsecondsS | |||
// "P"+(days+"D")?+"T"+(hours+"H")?+(minutes+"M")?+(seconds+"S")? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was a very manual change. I don't think the previous text was particularly readable...
@@ -148,13 +148,13 @@ SUITE(proxy_tests) | |||
TEST_FIXTURE(uri_address, | |||
http_proxy_with_credentials, | |||
"Ignore:Linux", | |||
"Github 53", | |||
"GitHub 53", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure what this is doing, it's possible it can't be applied... happy to drop if it's a problem...
@@ -45,7 +45,7 @@ listed below. | |||
|
|||
base64.cpp and base64.h | |||
|
|||
Copyright (C) 2004-2008 Ren� Nyffenegger | |||
Copyright (C) 2004-2008 Ren� Nyffenegger |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure what happened to this line. I blame VS Code
|
||
****** SHA1 Library (sha1/sha1.hpp) ****** | ||
sha1.hpp is a repackaging of the sha1.cpp and sha1.h files from the shallsha1 | ||
sha1.hpp is a repackaging of the sha1.cpp and sha1.h files from the smallsha1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't normally change files like this, but this typo seemed sufficiently problematic/worthy of correcting.
@@ -15,7 +15,7 @@ cpprestsdk (2.10.18) | |||
|
|||
cpprestsdk (2.10.17) | |||
* PR#1550 Fix year calculation for the last day of a leap year | |||
* PR#1523 Fix wrong linking of Apple Frameworks on MacOS | |||
* PR#1523 Fix wrong linking of Apple Frameworks on macOS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some projects don't like changing changelogs, I'm happy to drop this (or anything else) if it's a problem.
This PR corrects misspellings identified by the check-spelling action (which is an evolution of the script I used ages ago when I first made a PR here...).
The misspellings have been reported at https://github.com/jsoref/cpprestsdk/actions/runs/12292228454#summary-34302451221
The action reports that the changes in this PR would make it happy: https://github.com/jsoref/cpprestsdk/actions/runs/12292228864#summary-34302452484