Skip to content

Commit 2e82a7d

Browse files
authored
Merge pull request #330 from millerds/merge-shared-changes
Merge shared changes
2 parents cfae84b + 200211d commit 2e82a7d

File tree

6 files changed

+851
-453
lines changed

6 files changed

+851
-453
lines changed

.azure-devops/test.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ steps:
77
- task: CmdLine@2
88
inputs:
99
script: |
10-
1110
echo Setting WebView Type: ${{ parameters.webView }}
1211
call npx office-addin-dev-settings webview manifest.xml ${{ parameters.webView }}
1312
call npx office-addin-dev-settings webview test/end-to-end/test-manifest.xml ${{ parameters.webView }}

.github/pull_request_template.md

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
Thank you for your pull request! Please provide the following information.
2+
3+
---
4+
5+
**Change Description**:
6+
7+
Describe what the PR is about.
8+
9+
1. **Do these changes impact any *npm scripts* commands (in package.json)?** (e.g., running 'npm run start')
10+
If Yes, briefly describe what is impacted.
11+
12+
13+
2. **Do these changes impact *VS Code debugging* options (launch.json)?**
14+
If Yes, briefly describe what is impacted.
15+
16+
17+
3. **Do these changes impact *template output*?** (e.g., add/remove file, update file location, update file contents)
18+
If Yes, briefly describe what is impacted.
19+
20+
21+
4. **Do these changes impact *documentation*?** (e.g., a tutorial on https://docs.microsoft.com/en-us/office/dev/add-ins/overview/office-add-ins)
22+
If Yes, briefly describe what is impacted.
23+
24+
25+
If you answered yes to any of these please do the following:
26+
> Include 'Rick-Kirkham' in the review
27+
> Make sure the README file is correct
28+
29+
**Validation/testing performed**:
30+
31+
Describe manual testing done.

0 commit comments

Comments
 (0)