forcedotcom / forcedotcom/code-analyzer

[Feature Request] Add Input Variable To Control PR Review Behavior

オープン
#1,965 コメント 3 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
TypeScript
スター
240
フォーク
52
平均マージ
1日 23時間
マージ済み PR(30日)
5

説明

### Product Area

The “Run Salesforce Code Analyzer” GitHub Action

### Your Need or Problem

Currently, if I want to leverage the new `num-sev{X}-violations-in-changed-files` output variables I have to provide a github token with read access. Doing so leads to the action submitting a review for my PR.

I don't want the action to leave a review, I would simply like to know if there are violations in my changed files so that I can take actions against that in a following step. This is a pain point.

### Your Desired Solution

I would like some sort of input variable that lets me disable the code review. This should default to false to ensure that no existing workflows are broken. This would allow me to make use of the newly introduced `*in-chaged-files` output variables _without_ having a a review submitted via the action itself.

Something along the lines of:

```yaml
disableReview:
description: 'Prevents the action from submitting a review even if the github token is provided'
required: false
type: 'boolean'
default: 'false'
```

### Alternatives Considered

_No response_

### Additional Context (Screenshots, Files, etc)

_No response_

### Workaround

_No response_

### Urgency

Moderate

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず、GitHub Action の input 定義と、pull-request review を送信するパスを特定します。新しいオプションの input、そのデフォルト値 false、および review を送信しなくても違反の出力を利用できるケースをカバーするテストを追加し、Action の既存のテストまたは workflow チェックを実行します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
github-actions, typescript
領域
ci-cd
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。