Abacn commented on code in PR #29549: URL: https://github.com/apache/beam/pull/29549#discussion_r1408507954
########## .github/workflows/README.md: ########## @@ -271,7 +271,6 @@ Please note that jobs with matrix need to have matrix element in the comment. Ex | Workflow name | Matrix | Trigger Phrase | Cron Status | |:-------------:|:------:|:--------------:|:-----------:| -| [ PostCommit BeamMetrics Publish ](https://github.com/apache/beam/actions/workflows/beam_PostCommit_BeamMetrics_Publish.yml) | N/A |`Run Beam Metrics Deployment`| [](https://github.com/apache/beam/actions/workflows/beam_PostCommit_BeamMetrics_Publish.yml?query=event%3Aschedule) Review Comment: BeamMetrics Publish and Website Publish are not really "postcommit" or test suites. It **deploys** the Beam website or **deploys** the community metrics dashboard. They should not have comment trigger (which could be used to attack beam website). So renamed them and move to "other" category. -- 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]
