Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: git checkout HEAD^2 step is no longer necessary #164

Merged
merged 1 commit into from
Jan 24, 2021

Conversation

bgianfo
Copy link
Contributor

@bgianfo bgianfo commented Jan 24, 2021

The CI builds now report this warning:

1 issue was detected with this workflow: git checkout HEAD^2 is no longer necessary.
Please remove this step as Code Scanning recommends analyzing the merge commit for best results

This step was recommended but is now unnecessary.

The CI builds now report this warning:

```
1 issue was detected with this workflow: git checkout HEAD^2 is no longer necessary.
Please remove this step as Code Scanning recommends analyzing the merge commit for best results
```

This step was recommended, but is now unnecessary apparently.
@bgianfo bgianfo added the bug Something isn't working label Jan 24, 2021
@bgianfo bgianfo added this to In Progress in Next Major Release (4.x) Jan 24, 2021
@bgianfo bgianfo merged commit 2ba03b3 into microsoft:master Jan 24, 2021
Next Major Release (4.x) automation moved this from In Progress to Done Jan 24, 2021
@bgianfo bgianfo deleted the fix-ci-warning branch January 24, 2021 11:41
number201724 pushed a commit to number201724/Detours that referenced this pull request Mar 5, 2021
The CI builds now report this warning:

```
1 issue was detected with this workflow: git checkout HEAD^2 is no longer necessary.
Please remove this step as Code Scanning recommends analyzing the merge commit for best results
```

This step was recommended, but is now unnecessary apparently.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant