potiuk commented on code in PR #799:
URL: https://github.com/apache/airflow-site/pull/799#discussion_r1226193545


##########
post-docs/add-back-references.py:
##########
@@ -22,14 +22,30 @@
 from urllib.request import urlopen
 import semver
 
+log = logging.getLogger(__name__)
+
 airflow_redirects_link = 
"https://raw.githubusercontent.com/apache/airflow/main/docs/apache-airflow/redirects.txt";
 helm_redirects_link = 
"https://raw.githubusercontent.com/apache/airflow/main/docs/helm-chart/redirects.txt";
-providers_redirect_link = "populate-this"
 
 docs_archive_path = "../docs-archive"
 airflow_docs_path = docs_archive_path + "/apache-airflow"
 helm_docs_path = docs_archive_path + "/helm-chart"
-providers_docs_path = docs_archive_path + "/apache-airflow-providers"
+
+# if any new provider is added with redirects, add it to this list

Review Comment:
   Why: the less chance we have to forget to update something, the better. This 
is for example why in my in-progress PR about sbom generation, I initially had 
a hard-coded list of Airflow versions - but eventually I settled on retriving 
them from tags in the git remote "apache/airflow" repo:
    
https://github.com/apache/airflow/pull/31835/files#diff-54ea68b5e1e61ac0437df0ab7d99dc508595323b11847a957931e0d9007962d7R1241



-- 
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