Skip to content

Commit 9502e4e

Browse files
authored
BRE-609/workflow-changes (#266)
## 🎟️ Tracking [BRE-609](https://bitwarden.atlassian.net/browse/BRE-609) ## 📔 Objective update workflows to be compliant with pull_request_target workflow linter rule ahead of changing rule log level to error ## ⏰ Reminders before review - Contributor guidelines followed - All formatters and local linters executed and passed - Written new unit and / or integration tests where applicable - Protected functional changes with optionality (feature flags) - Used internationalization (i18n) for all UI strings - CI builds passed - Communicated to DevOps any deployment requirements - Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team ## 🦮 Reviewer guidelines <!-- Suggested interactions but feel free to use (or not) as you desire! --> - 👍 (`:+1:`) or similar for great changes - 📝 (`:memo:`) or ℹ️ (`:information_source:`) for notes or general info - ❓ (`:question:`) for questions - 🤔 (`:thinking:`) or 💭 (`:thought_balloon:`) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion - 🎨 (`:art:`) for suggestions / improvements - ❌ (`:x:`) or ⚠️ (`:warning:`) for more significant problems or concerns needing attention - 🌱 (`:seedling:`) or ♻️ (`:recycle:`) for future improvements or indications of technical debt - ⛏ (`:pick:`) for minor or nitpick changes [BRE-609]: https://bitwarden.atlassian.net/browse/BRE-609?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
1 parent 58bf7d4 commit 9502e4e

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

.github/workflows/scan.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,14 @@ on:
77
- "main"
88
- "rc"
99
- "hotfix-rc"
10+
pull_request:
11+
types: [opened, synchronize, reopened]
12+
branches-ignore:
13+
- main
1014
pull_request_target:
11-
types: [opened, synchronize]
15+
types: [opened, synchronize, reopened]
16+
branches:
17+
- "main"
1218

1319
jobs:
1420
check-run:

0 commit comments

Comments
 (0)