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.

Sign up for GitHub

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

Jump to bottom

[ReleasePrep][2020.08.07] RI of dev into master #8147

Merged
merged 8 commits into from
Aug 8, 2020
Merged

[ReleasePrep][2020.08.07] RI of dev into master #8147

merged 8 commits into from
Aug 8, 2020

Conversation

loic-sharma
Copy link
Contributor

Deployment work item: #8145

t-mog-msft and others added 7 commits July 30, 2020 18:23
@t-mog-msft
Add functional tests on Gallery for Advanced Search (#8115)
94e4009 
* Add some functional tests

* Fix PR comments

* Remove unnecessary pivot

* Fix PR comments

* Encoded colon
@joelverhagen
Use protected configuration provider instead of reflection (WITHOUT D…
c2fbcee 
…I) (#8132)

Address #8121
Related to https://github.com/NuGet/Engineering/issues/3206
@scottbommarito
Add command to compare PackageVulnerabilities to GitHub API (#7706)
df53bf8 
Add command to compare PackageVulnerabilities to GitHub API
@loic-sharma
Add note on functional tests' requirements (#8134)
316888a
@drewgillies
Unbreak GitHubVulnerabilities2Db dependency injection (#8139)
a4403ae 
unbreak ghv2db dependency injection - use content service fake
@skofman1
Fix for: Malicious OData requests trigger exception monitoring (#8141)
37f3885 
* Remove exception logging for OData format exception in Get
@drewgillies
Update range's vulnerability before entity update to fix index unique…
4c77f91 
…ness error (#8140)
@loic-sharma loic-sharma self-assigned this Aug 7, 2020
@drewgillies drewgillies mentioned this pull request Aug 7, 2020
Closed
12 tasks
@loic-sharma
Copy link
Contributor Author

I've verified @joelverhagen machine's key changes on DEV. @drewgillies has verified his changes and @scottbommarito's on DEV

skofman1
skofman1 approved these changes Aug 7, 2020
View reviewed changes
Copy link
Contributor

@skofman1 skofman1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified my change on dev

@skofman1
When V1 Odata requests OrderBy is invalid, return 400 (#8144)
6c5734e 
* When V1 Odata requests OrderBy is invalid, return 400

* When V2 Odata requests OrderBy is invalid, return 400
@loic-sharma loic-sharma merged commit b3c174e into master Aug 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skofman1 skofman1 skofman1 approved these changes

@t-mog-msft t-mog-msft t-mog-msft approved these changes

@joelverhagen joelverhagen Awaiting requested review from joelverhagen

@drewgillies drewgillies Awaiting requested review from drewgillies

@agr agr Awaiting requested review from agr

@cristinamanum cristinamanum Awaiting requested review from cristinamanum

@dannyjdev dannyjdev Awaiting requested review from dannyjdev

@lyndaidaii lyndaidaii Awaiting requested review from lyndaidaii

@ryuyu ryuyu Awaiting requested review from ryuyu

@shishirx34 shishirx34 Awaiting requested review from shishirx34

@xavierdecoster xavierdecoster Awaiting requested review from xavierdecoster

@zhhyu zhhyu Awaiting requested review from zhhyu

Assignees

@loic-sharma loic-sharma

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@loic-sharma @skofman1 @t-mog-msft @joelverhagen @scottbommarito @drewgillies