Skip to content

Commit f59e175

Browse files
koroskoros
authored
Merge branch 'dev' into gkoros/fix-compilation-warnings
2 parents 021a1fb + 437cade commit f59e175

File tree

4 files changed

+663
-677
lines changed

4 files changed

+663
-677
lines changed

‎.github/policies/msgraph-sdk-javascript-branch-protection.yml

+207
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,207 @@
1+
# Copyright (c) Microsoft Corporation.
2+
# Licensed under the MIT License.
3+
4+
# File initially created using https://github.com/MIchaelMainer/policyservicetoolkit/blob/main/branch_protection_export.ps1.
5+
6+
name: msgraph-sdk-javascript-branch-protection
7+
description: Branch protection policy for the msgraph-sdk-javascript repository
8+
resource: repository
9+
configuration:
10+
branchProtectionRules:
11+
12+
- branchNamePattern: dev
13+
# This branch pattern applies to the following branches as of 06/12/2023 10:31:16:
14+
# dev
15+
16+
# Specifies whether this branch can be deleted. boolean
17+
allowsDeletions: false
18+
# Specifies whether forced pushes are allowed on this branch. boolean
19+
allowsForcePushes: false
20+
# Specifies whether new commits pushed to the matching branches dismiss pull request review approvals. boolean
21+
dismissStaleReviews: true
22+
# Specifies whether admins can overwrite branch protection. boolean
23+
isAdminEnforced: false
24+
# Indicates whether "Require a pull request before merging" is enabled. boolean
25+
requiresPullRequestBeforeMerging: true
26+
# Specifies the number of pull request reviews before merging. int (0-6). Should be null/empty if PRs are not required
27+
requiredApprovingReviewsCount: 1
28+
# Require review from Code Owners. Requires requiredApprovingReviewsCount. boolean
29+
requireCodeOwnersReview: true
30+
# Are commits required to be signed. boolean. TODO: all contributors must have commit signing on local machines.
31+
requiresCommitSignatures: false
32+
# Are conversations required to be resolved before merging? boolean
33+
requiresConversationResolution: false
34+
# Are merge commits prohibited from being pushed to this branch. boolean
35+
requiresLinearHistory: false
36+
# Required status checks to pass before merging. Values can be any string, but if the value does not correspond to any existing status check, the status check will be stuck on pending for status since nothing exists to push an actual status
37+
requiredStatusChecks:
38+
- build (12.x)
39+
- build (14.x)
40+
- build (16.x)
41+
# Require branches to be up to date before merging. Requires requiredStatusChecks. boolean
42+
requiresStrictStatusChecks: true
43+
# Indicates whether there are restrictions on who can push. boolean. Should be set with whoCanPush.
44+
restrictsPushes: false
45+
# Restrict who can dismiss pull request reviews. boolean
46+
restrictsReviewDismissals: false
47+
48+
- branchNamePattern: master
49+
# This branch pattern applies to the following branches as of 06/12/2023 10:31:16:
50+
# master
51+
52+
# Specifies whether this branch can be deleted. boolean
53+
allowsDeletions: false
54+
# Specifies whether forced pushes are allowed on this branch. boolean
55+
allowsForcePushes: false
56+
# Specifies whether new commits pushed to the matching branches dismiss pull request review approvals. boolean
57+
dismissStaleReviews: true
58+
# Specifies whether admins can overwrite branch protection. boolean
59+
isAdminEnforced: false
60+
# Indicates whether "Require a pull request before merging" is enabled. boolean
61+
requiresPullRequestBeforeMerging: true
62+
# Specifies the number of pull request reviews before merging. int (0-6). Should be null/empty if PRs are not required
63+
requiredApprovingReviewsCount: 1
64+
# Require review from Code Owners. Requires requiredApprovingReviewsCount. boolean
65+
requireCodeOwnersReview: true
66+
# Are commits required to be signed. boolean. TODO: all contributors must have commit signing on local machines.
67+
requiresCommitSignatures: false
68+
# Are conversations required to be resolved before merging? boolean
69+
requiresConversationResolution: false
70+
# Are merge commits prohibited from being pushed to this branch. boolean
71+
requiresLinearHistory: false
72+
# Required status checks to pass before merging. Values can be any string, but if the value does not correspond to any existing status check, the status check will be stuck on pending for status since nothing exists to push an actual status
73+
requiredStatusChecks:
74+
# Require branches to be up to date before merging. Requires requiredStatusChecks. boolean
75+
requiresStrictStatusChecks: true
76+
# Indicates whether there are restrictions on who can push. boolean. Should be set with whoCanPush.
77+
restrictsPushes: false
78+
# Restrict who can dismiss pull request reviews. boolean
79+
restrictsReviewDismissals: false
80+
81+
- branchNamePattern: 3.0.0
82+
# This branch pattern applies to the following branches as of 06/12/2023 10:31:16:
83+
# 3.0.0
84+
85+
# Specifies whether this branch can be deleted. boolean
86+
allowsDeletions: false
87+
# Specifies whether forced pushes are allowed on this branch. boolean
88+
allowsForcePushes: false
89+
# Specifies whether new commits pushed to the matching branches dismiss pull request review approvals. boolean
90+
dismissStaleReviews: false
91+
# Specifies whether admins can overwrite branch protection. boolean
92+
isAdminEnforced: false
93+
# Require review from Code Owners. Requires requiredApprovingReviewsCount. boolean
94+
requireCodeOwnersReview: false
95+
# Are commits required to be signed. boolean. TODO: all contributors must have commit signing on local machines.
96+
requiresCommitSignatures: false
97+
# Are conversations required to be resolved before merging? boolean
98+
requiresConversationResolution: false
99+
# Are merge commits prohibited from being pushed to this branch. boolean
100+
requiresLinearHistory: false
101+
# Require branches to be up to date before merging. Requires requiredStatusChecks. boolean
102+
requiresStrictStatusChecks: true
103+
# Indicates whether there are restrictions on who can push. boolean. Should be set with whoCanPush.
104+
restrictsPushes: false
105+
# Restrict who can dismiss pull request reviews. boolean
106+
restrictsReviewDismissals: false
107+
108+
- branchNamePattern: support/v2
109+
# This branch pattern applies to the following branches as of 06/12/2023 10:31:16:
110+
# support/v2
111+
112+
# Specifies whether this branch can be deleted. boolean
113+
allowsDeletions: false
114+
# Specifies whether forced pushes are allowed on this branch. boolean
115+
allowsForcePushes: false
116+
# Specifies whether new commits pushed to the matching branches dismiss pull request review approvals. boolean
117+
dismissStaleReviews: false
118+
# Specifies whether admins can overwrite branch protection. boolean
119+
isAdminEnforced: false
120+
# Indicates whether "Require a pull request before merging" is enabled. boolean
121+
requiresPullRequestBeforeMerging: true
122+
# Specifies the number of pull request reviews before merging. int (0-6). Should be null/empty if PRs are not required
123+
requiredApprovingReviewsCount: 1
124+
# Require review from Code Owners. Requires requiredApprovingReviewsCount. boolean
125+
requireCodeOwnersReview: false
126+
# Are commits required to be signed. boolean. TODO: all contributors must have commit signing on local machines.
127+
requiresCommitSignatures: false
128+
# Are conversations required to be resolved before merging? boolean
129+
requiresConversationResolution: false
130+
# Are merge commits prohibited from being pushed to this branch. boolean
131+
requiresLinearHistory: false
132+
# Require branches to be up to date before merging. Requires requiredStatusChecks. boolean
133+
requiresStrictStatusChecks: true
134+
# Indicates whether there are restrictions on who can push. boolean. Should be set with whoCanPush.
135+
restrictsPushes: false
136+
# Restrict who can dismiss pull request reviews. boolean
137+
restrictsReviewDismissals: false
138+
139+
- branchNamePattern: v2/dev
140+
# This branch pattern applies to the following branches as of 06/12/2023 10:31:16:
141+
# v2/dev
142+
143+
# Specifies whether this branch can be deleted. boolean
144+
allowsDeletions: false
145+
# Specifies whether forced pushes are allowed on this branch. boolean
146+
allowsForcePushes: false
147+
# Specifies whether new commits pushed to the matching branches dismiss pull request review approvals. boolean
148+
dismissStaleReviews: false
149+
# Specifies whether admins can overwrite branch protection. boolean
150+
isAdminEnforced: false
151+
# Indicates whether "Require a pull request before merging" is enabled. boolean
152+
requiresPullRequestBeforeMerging: true
153+
# Specifies the number of pull request reviews before merging. int (0-6). Should be null/empty if PRs are not required
154+
requiredApprovingReviewsCount: 1
155+
# Require review from Code Owners. Requires requiredApprovingReviewsCount. boolean
156+
requireCodeOwnersReview: false
157+
# Are commits required to be signed. boolean. TODO: all contributors must have commit signing on local machines.
158+
requiresCommitSignatures: false
159+
# Are conversations required to be resolved before merging? boolean
160+
requiresConversationResolution: false
161+
# Are merge commits prohibited from being pushed to this branch. boolean
162+
requiresLinearHistory: false
163+
# Required status checks to pass before merging. Values can be any string, but if the value does not correspond to any existing status check, the status check will be stuck on pending for status since nothing exists to push an actual status
164+
requiredStatusChecks:
165+
# Require branches to be up to date before merging. Requires requiredStatusChecks. boolean
166+
requiresStrictStatusChecks: false
167+
# Indicates whether there are restrictions on who can push. boolean. Should be set with whoCanPush.
168+
restrictsPushes: false
169+
# Restrict who can dismiss pull request reviews. boolean
170+
restrictsReviewDismissals: false
171+
172+
- branchNamePattern: main
173+
# This branch pattern applies to the following branches as of 06/12/2023 10:31:16:
174+
# main
175+
176+
# Specifies whether this branch can be deleted. boolean
177+
allowsDeletions: false
178+
# Specifies whether forced pushes are allowed on this branch. boolean
179+
allowsForcePushes: false
180+
# Specifies whether new commits pushed to the matching branches dismiss pull request review approvals. boolean
181+
dismissStaleReviews: false
182+
# Specifies whether admins can overwrite branch protection. boolean
183+
isAdminEnforced: false
184+
# Indicates whether "Require a pull request before merging" is enabled. boolean
185+
requiresPullRequestBeforeMerging: true
186+
# Specifies the number of pull request reviews before merging. int (0-6). Should be null/empty if PRs are not required
187+
requiredApprovingReviewsCount: 1
188+
# Require review from Code Owners. Requires requiredApprovingReviewsCount. boolean
189+
requireCodeOwnersReview: false
190+
# Are commits required to be signed. boolean. TODO: all contributors must have commit signing on local machines.
191+
requiresCommitSignatures: false
192+
# Are conversations required to be resolved before merging? boolean
193+
requiresConversationResolution: false
194+
# Are merge commits prohibited from being pushed to this branch. boolean
195+
requiresLinearHistory: false
196+
# Required status checks to pass before merging. Values can be any string, but if the value does not correspond to any existing status check, the status check will be stuck on pending for status since nothing exists to push an actual status
197+
requiredStatusChecks:
198+
- build (14.x)
199+
- build (16.x)
200+
- build (12.x)
201+
# Require branches to be up to date before merging. Requires requiredStatusChecks. boolean
202+
requiresStrictStatusChecks: true
203+
# Indicates whether there are restrictions on who can push. boolean. Should be set with whoCanPush.
204+
restrictsPushes: false
205+
# Restrict who can dismiss pull request reviews. boolean
206+
restrictsReviewDismissals: false
207+

‎docs/CreatingClientInstance.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ The Microsoft Graph JavaScript Client Library has an adapter implementation for
3737

3838
> Learn how to [create an instance of TokenCredentialAuthenticationProvider](./TokenCredentialAuthenticationProvider.md).
3939
40-
- ([AuthCodeMSALBrowserAuthenticationProvider](../src/authentication/msal/ImplicitMSALAuthenticationProvider.ts)) for [msal-browser](https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/lib/msal-browser) (Microsoft Authentication Library) which takes care of getting the `accessToken`. `msal-browser` library does not ship with this library, user has to include it externally.
40+
- ([AuthCodeMSALBrowserAuthenticationProvider](../src/authentication/msal-browser/AuthCodeMSALBrowserAuthenticationProvider.ts) for [msal-browser](https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/lib/msal-browser) (Microsoft Authentication Library) which takes care of getting the `accessToken`. `msal-browser` library does not ship with this library, user has to include it externally.
4141

4242
> Learn how to [create an instance of AuthCodeMSALBrowserAuthenticationProvider](./AuthCodeMSALBrowserAuthenticationProvider.md).
4343

0 commit comments

Comments
 (0)