Skip to content

Commit c09c501

Browse files
authored
Merge pull request advanced-security-demo#7 from octodemo/code-scanning-pr-experience
Code scanning PR experience
2 parents ab2f977 + c70ae5b commit c09c501

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

README.md

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ This tutorial with use CodeQL Analysis with Code Scanning in order to search for
1818

1919
Begin by [forking this repo](https://docs.github.com/en/free-pro-team@latest/github/getting-started-with-github/fork-a-repo).
2020

21-
Make sure you uncheck "Copy the main branch only"
21+
NOTE: Make sure you uncheck "Copy the `main` branch only"
2222

2323
<img src="images/17-fork-repo.png" width="70%"/>
2424

@@ -215,7 +215,12 @@ A branch called `new-feature` introduces a new feature but also security vulnera
215215

216216
1. Go to the Pull Request tab
217217
2. Select "New Pull Request"
218-
3. Create the PR with `base: main` and `compare: new-feature`
218+
3. Create the PR with
219+
- `base repository: <YOUR FORK>`
220+
- `head repository: <YOUR FORK>`
221+
- `base: main`
222+
- `compare: new-feature`
223+
4. _If you don't see the `new-feature` branch, change the `head repository: octodemo/advanced-security-python`_
219224

220225
<img src="images/18-create-vulnerable-pr.png" width="80%"/>
221226

@@ -233,7 +238,7 @@ Notice that Code Scanning has detected that this Pull Request will introduce 2 m
233238

234239
<img src="images/19-vulnerabiltliy-detail.png" width="80%"/>
235240

236-
### 'Files Changed' tab
241+
#### 'Files Changed' tab
237242

238243
Click on the "Files Changed" tab of the PR. Scroll down and notice the Advanced Security annotations for new vulnerabilities.
239244

0 commit comments

Comments
 (0)