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

Repo File Sync: synced file(s) with microsoft/mu_devops #300

Merged
merged 1 commit into from
Oct 2, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .azurepipelines/MuDevOpsWrapper.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ resources:
type: github
endpoint: microsoft
name: microsoft/mu_devops
ref: refs/tags/v10.0.0
ref: refs/tags/v12.0.2

parameters:
- name: do_ci_build
Expand Down Expand Up @@ -82,7 +82,7 @@ parameters:
jobs:
- template: Jobs/PrGate.yml@mu_devops
parameters:
linux_container_image: ghcr.io/microsoft/mu_devops/ubuntu-22-build:0e124c1
linux_container_image: ghcr.io/microsoft/mu_devops/ubuntu-22-build:d1e4ff1
${{ if eq(parameters.rust_build, true) }}:
linux_container_options: --security-opt seccomp=unconfined
do_ci_build: ${{ parameters.do_ci_build }}
Expand All @@ -105,7 +105,7 @@ jobs:

container:

image: ghcr.io/microsoft/mu_devops/ubuntu-22-build:0e124c1
image: ghcr.io/microsoft/mu_devops/ubuntu-22-build:d1e4ff1
options: --user root --name mu_devops_build_container --security-opt seccomp=unconfined

steps:
Expand Down
29 changes: 3 additions & 26 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,41 +1,18 @@
# Preface

Please ensure you have read the [contribution docs](https://github.com/microsoft/mu/blob/master/CONTRIBUTING.md) prior
to submitting the pull request. In particular,
[pull request guidelines](https://github.com/microsoft/mu/blob/master/CONTRIBUTING.md#pull-request-best-practices).

## Description

<_Please include a description of the change and why this change was made._>
<_Include a description of the change and why this change was made._>

For each item, place an "x" in between `[` and `]` if true. Example: `[x]`.
_(you can also check items in the GitHub UI)_
For details on how to complete these options and their meaning refer to [CONTRIBUTING.md](https://github.com/microsoft/mu/blob/HEAD/CONTRIBUTING.md).

- [ ] Impacts functionality?
- **Functionality** - Does the change ultimately impact how firmware functions?
- Examples: Add a new library, publish a new PPI, update an algorithm, ...
- [ ] Impacts security?
- **Security** - Does the change have a direct security impact on an application,
flow, or firmware?
- Examples: Crypto algorithm change, buffer overflow fix, parameter
validation improvement, ...
- [ ] Breaking change?
- **Breaking change** - Will anyone consuming this change experience a break
in build or boot behavior?
- Examples: Add a new library class, move a module to a different repo, call
a function in a new library class in a pre-existing module, ...
- [ ] Includes tests?
- **Tests** - Does the change include any explicit test code?
- Examples: Unit tests, integration tests, robot tests, ...
- [ ] Includes documentation?
- **Documentation** - Does the change contain explicit documentation additions
outside direct code modifications (and comments)?
- Examples: Update readme file, add feature readme file, link to documentation
on an a separate Web page, ...

## How This Was Tested

<_Please describe the test(s) that were run to verify the changes._>
<_Describe the test(s) that were run to verify the changes._>

## Integration Instructions

Expand Down
2 changes: 1 addition & 1 deletion .github/release-draft-config-n-1.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
name-template: 'v$RESOLVED_VERSION'
tag-template: 'v$RESOLVED_VERSION'

commitish: refs/heads/release/202302
commitish: refs/heads/release/202311
filter-by-commitish: true

template: |
Expand Down
2 changes: 1 addition & 1 deletion .github/release-draft-config-n.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
name-template: 'v$RESOLVED_VERSION'
tag-template: 'v$RESOLVED_VERSION'

commitish: refs/heads/release/202311
commitish: refs/heads/release/202405
filter-by-commitish: true

template: |
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/auto-approve.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,5 +29,5 @@ jobs:

if: |
github.event.pull_request.user.login == 'dependabot[bot]' || github.event.pull_request.user.login == 'uefibot'
uses: microsoft/mu_devops/.github/workflows/AutoApprover.yml@v10.0.0
uses: microsoft/mu_devops/.github/workflows/AutoApprover.yml@v12.0.2
secrets: inherit
2 changes: 1 addition & 1 deletion .github/workflows/auto-merge.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,5 +32,5 @@ jobs:

if: |
github.event.pull_request.user.login == 'dependabot[bot]' || github.event.pull_request.user.login == 'uefibot'
uses: microsoft/mu_devops/.github/workflows/AutoMerger.yml@v10.0.0
uses: microsoft/mu_devops/.github/workflows/AutoMerger.yml@v12.0.2
secrets: inherit
2 changes: 2 additions & 0 deletions .github/workflows/codeql.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,12 @@ on:
branches:
- main
- release/*
- dev/*
pull_request:
branches:
- main
- release/*
- dev/*
paths-ignore:
- '!**.c'
- '!**.h'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/issue-assignment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,4 +23,4 @@ jobs:
contents: read
issues: write

uses: microsoft/mu_devops/.github/workflows/IssueAssignment.yml@v10.0.0
uses: microsoft/mu_devops/.github/workflows/IssueAssignment.yml@v12.0.2
2 changes: 1 addition & 1 deletion .github/workflows/label-issues.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,4 +36,4 @@ jobs:
contents: read
pull-requests: write

uses: microsoft/mu_devops/.github/workflows/Labeler.yml@v10.0.0
uses: microsoft/mu_devops/.github/workflows/Labeler.yml@v12.0.2
2 changes: 1 addition & 1 deletion .github/workflows/label-sync.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,4 +28,4 @@ jobs:
permissions:
issues: write

uses: microsoft/mu_devops/.github/workflows/LabelSyncer.yml@v10.0.0
uses: microsoft/mu_devops/.github/workflows/LabelSyncer.yml@v12.0.2
4 changes: 2 additions & 2 deletions .github/workflows/release-draft.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ name: Update Release Draft
on:
push:
branches:
- release/202311
- release/202405

jobs:
draft:
Expand All @@ -32,5 +32,5 @@ jobs:
contents: write
pull-requests: write

uses: microsoft/mu_devops/.github/workflows/ReleaseDrafter.yml@v10.0.0
uses: microsoft/mu_devops/.github/workflows/ReleaseDrafter.yml@v12.0.2
secrets: inherit
2 changes: 1 addition & 1 deletion .github/workflows/stale.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,4 +29,4 @@ jobs:
issues: write
pull-requests: write

uses: microsoft/mu_devops/.github/workflows/Stale.yml@v10.0.0
uses: microsoft/mu_devops/.github/workflows/Stale.yml@v12.0.2
2 changes: 1 addition & 1 deletion .github/workflows/triage-issues.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,4 +23,4 @@ jobs:
permissions:
issues: write

uses: microsoft/mu_devops/.github/workflows/IssueTriager.yml@v10.0.0
uses: microsoft/mu_devops/.github/workflows/IssueTriager.yml@v12.0.2
53 changes: 52 additions & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ submitted in the issues section.
## Security Vulnerabilities

Please review the repos `Security Policy` but in general every Project Mu repo has `Private vulnerability reporting`
enabled. Please use the security tab to report a potential issue.
enabled. Please use the security tab to report a potential issue.

### Identify Where to Report

Expand Down Expand Up @@ -63,6 +63,57 @@ configuration files. To aid maintainers in reviewing your code, we suggest adher
* If the contribution logically be broken up into separate pull requests that independently build and function
successfully, do use multiple pull requests.

#### Pull Request Description Checkboxes

Project Mu pull requests autopopulate a PR description from a template in most repositories. You should:

1. **Replace** this text with an actual descrption:

```txt
<_Include a description of the change and why this change was made._>
```

2. **Remove** this line of instructions so the PR description shows cleanly in release notes:

`"For details on how to complete these options and their meaning refer to [CONTRIBUTING.md](https://github.com/microsoft/mu/blob/HEAD/CONTRIBUTING.md)."`

3. For each checkbox in the PR description, **place an "x"** in between `[` and `]` if true. Example: `[x]`.
_(you can also check items in the GitHub UI)_

* **[] Impacts functionality?**
* **Functionality** - Does the change ultimately impact how firmware functions?
* Examples: Add a new library, publish a new PPI, update an algorithm, ...
* **[] Impacts security?**
* **Security** - Does the change have a direct security impact on an application,
flow, or firmware?
* Examples: Crypto algorithm change, buffer overflow fix, parameter
validation improvement, ...
* **[] Breaking change?**
* **Breaking change** - Will anyone consuming this change experience a break
in build or boot behavior?
* Examples: Add a new library class, move a module to a different repo, call
a function in a new library class in a pre-existing module, ...
* [] **Includes tests?**
* **Tests** - Does the change include any explicit test code?
* Examples: Unit tests, integration tests, robot tests, ...
* [] **Includes documentation?**
* **Documentation** - Does the change contain explicit documentation additions
outside direct code modifications (and comments)?
* Examples: Update readme file, add feature readme file, link to documentation
on an a separate Web page, ...

4. **Replace** this text as instructed:

```txt
<_Describe the test(s) that were run to verify the changes._>
```

5. **Replace** this text as instructed:

```txt
<_Describe how these changes should be integrated. Use N/A if nothing is required._>
```

#### Code Categories

To keep code digestible, you may consider breaking large pull requests into three categories of commits within the pull
Expand Down
Loading