[PATCH] D155081: Specify the developer policy around links to external resources

2023-08-14 Thread Tanya Lattner via Phabricator via cfe-commits
tonic added a comment. @lattner Please review. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155081/new/ https://reviews.llvm.org/D155081 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/

[PATCH] D155081: Specify the developer policy around links to external resources

2023-07-17 Thread Tanya Lattner via Phabricator via cfe-commits
tonic added a comment. As I mentioned on the Discourse thread, but if this policy change is made, there should be some discussion about if this applies retroactively. Making a note here, because it probably should be included in the changes to Developer Policy if the radar links are not removed

[PATCH] D132550: Changes to code ownership in clang and clang-tidy

2022-08-29 Thread Tanya Lattner via Phabricator via cfe-commits
tonic added a comment. Moving to GitHub PRs is the path forward. Let's save ourselves some work and add them now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132550/new/ https://reviews.llvm.org/D132550 _

[PATCH] D128766: Update references to Discourse instead of the mailing lists.

2022-07-08 Thread Tanya Lattner via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGeb1ffd817c3c: Update references to Discourse instead of the mailing lists. (authored by tonic). Changed prior to commit: https://reviews.llvm.org/

[PATCH] D128766: Update references to Discourse instead of the mailing lists.

2022-07-01 Thread Tanya Lattner via Phabricator via cfe-commits
tonic updated this revision to Diff 441800. tonic added a comment. Fix typos. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128766/new/ https://reviews.llvm.org/D128766 Files: clang/docs/ExternalClangExamples.rst clang/www/get_involved.html

[PATCH] D128766: Update references to Discourse instead of the mailing lists.

2022-06-28 Thread Tanya Lattner via Phabricator via cfe-commits
tonic created this revision. Herald added a project: All. tonic requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, sstefan1. Herald added a project: clang. Update the references to the old Mailman mailing lists to point to Discourse forum

[PATCH] D121078: Replace links to archived mailing lists by links to Discourse forums

2022-04-23 Thread Tanya Lattner via Phabricator via cfe-commits
tonic added a comment. In D121078#3402469 , @aaron.ballman wrote: > In D121078#3400810 , @SimplyDanny > wrote: > >> @tonic, can you please have another look? > > If @tonic has any additional feedback, we can han

[PATCH] D115907: [misexpect] Re-implement MisExpect Diagnostics

2022-03-28 Thread Tanya Lattner via Phabricator via cfe-commits
tonic added a comment. This is breaking the docs build with the following warning: "MisExpect.rst:document isn't included in any toctree." By default, docs warnings are treated as errors. Can you fix this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.

[PATCH] D121078: Replace links to archived mailing lists by links to Discourse forums

2022-03-08 Thread Tanya Lattner via Phabricator via cfe-commits
tonic added a comment. In D121078#3367950 , @aaron.ballman wrote: > In D121078#3367289 , @tonic wrote: > >> In D121078#3366825 , >> @aaron.ballman wrote: >> >>> In D1210

[PATCH] D121078: Replace links to archived mailing lists by links to Discourse forums

2022-03-08 Thread Tanya Lattner via Phabricator via cfe-commits
tonic added a comment. In D121078#3366825 , @aaron.ballman wrote: > In D121078#3366025 , @tonic wrote: > >> In D121078#3365542 , @SimplyDanny >> wrote: >> >>> In D121078

[PATCH] D121078: Replace links to archived mailing lists by links to Discourse forums

2022-03-07 Thread Tanya Lattner via Phabricator via cfe-commits
tonic added a comment. In D121078#3365542 , @SimplyDanny wrote: > In D121078#3363856 , @aaron.ballman > wrote: > >> I think we need to retain *some* references to the existing mailing list >> archives. The migr

[PATCH] D121078: Replace links to archived mailing lists by links to Discourse forums

2022-03-07 Thread Tanya Lattner via Phabricator via cfe-commits
tonic requested changes to this revision. tonic added a comment. Thank you so much for doing this! I have added some comments inline that I would like changed. Comment at: clang/README.txt:22 If you have questions or comments about Clang, a great place to discuss them is -on

[PATCH] D112730: [clang-tidy] Add AUTOSAR module

2022-02-11 Thread Tanya Lattner via Phabricator via cfe-commits
tonic added a comment. Unfortunately our legal counsel has advised that this patch should not be included into LLVM because the exact patent burden is not disclosed. Therefore, we can not accept this patch into LLVM. I am very sorry to have to report this disappointing news. CHANGES SINCE LAS

[PATCH] D36836: [clang-tidy] Implement sonarsource-function-cognitive-complexity check

2020-09-24 Thread Tanya Lattner via Phabricator via cfe-commits
tonic added a comment. Licensing questions are best sent to the bo...@llvm.org mailing address. Can you reach out there? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D36836/new/ https://reviews.llvm.org/D36836