kacpermuda commented on code in PR #37075:
URL: https://github.com/apache/airflow/pull/37075#discussion_r1484333085


##########
docs/spelling_wordlist.txt:
##########
@@ -122,6 +122,7 @@ avp
 Avro
 avro
 aws
+AwsBaseAsyncHook

Review Comment:
   @potiuk That makes sense, I will adjust the class names that were missing 
backticks, so that i don't have to whitelist them in spellchecking. 
   
   That leaves me with the question: if we start including deprecation message 
in the docs, when a developer forgets to put those backticks in the deprecation 
message, the spellcheck will fail and it will be to the person accepting the PR 
to make sure that they do not add those words to the whitelist but put 
backticks around them? Is there any way to add a clear information to the 
developer about why the spellcheck failed in this scenario so that he can 
correct himself ?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to