[GitHub] [airflow] eladkal commented on pull request #30541: Organize Amazon provider docs index

2023-04-14 Thread via GitHub
eladkal commented on PR #30541: URL: https://github.com/apache/airflow/pull/30541#issuecomment-1508378026 > Looks like one more `../' missing but other than that - LGTM. Glad that you sorted out AWS vs. Amazon vs. ... namings :) Yes due to https://github.com/apache/airflow/pull/28338

[GitHub] [airflow] eladkal commented on pull request #30541: Organize Amazon provider docs index

2023-04-13 Thread via GitHub
eladkal commented on PR #30541: URL: https://github.com/apache/airflow/pull/30541#issuecomment-1507373391 > but I think we need to stick with the proper names so that there is brand recognition for our users. NP. will modify accordingly -- This is an automated message from the Apac

[GitHub] [airflow] eladkal commented on pull request #30541: Organize Amazon provider docs index

2023-04-13 Thread via GitHub
eladkal commented on PR #30541: URL: https://github.com/apache/airflow/pull/30541#issuecomment-1507308089 @ferruzzi according to that link MWAA is not an official name :) IMHO it's very very annoying to go through a non sorted list so I try to minimize it as much as I can but I don't feel

[GitHub] [airflow] eladkal commented on pull request #30541: Organize Amazon provider docs index

2023-04-13 Thread via GitHub
eladkal commented on PR #30541: URL: https://github.com/apache/airflow/pull/30541#issuecomment-1506784544 Updated: ![aws](https://user-images.githubusercontent.com/45845474/231742004-bd281f8a-f4e1-4a17-9039-803bdd7a1e1f.gif) -- This is an automated message from the Apache Git S

[GitHub] [airflow] eladkal commented on pull request #30541: Organize Amazon provider docs index

2023-04-08 Thread via GitHub
eladkal commented on PR #30541: URL: https://github.com/apache/airflow/pull/30541#issuecomment-1500883895 > The A in AWS already stands for Amazon, “Amazon AWS” simply does not make sense. I agree but this is the current title so I mentioned it... -- This is an automated message fr