-
Notifications
You must be signed in to change notification settings - Fork 11
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
AF-3815 Add OSS licenses #5
Conversation
Merge Requirements Met ✅Request Rosie to automerge this pull request by including @Workiva/release-management-p in a comment.
Note: This is a shortened report. Click here to view Rosie's full evaluation. |
Security InsightsNo security relevant content was detected by automated scans. Action Items
Questions or Comments? Reach out on Slack: #support-infosec. |
889e800
to
2c2d517
Compare
+1 |
@Workiva/release-management-p |
@georgelesica-wf I will not merge this because:
|
+1 |
+1 from RM |
Description
Adds the OSS licenses.
Also adds a change in 8e35898 (sorting the file paths) to ensure a deterministic order of iteration of the files returned from a
FileQuery
. Without this, tests were failing intermittently on CI and it's probably useful/logical to iterate over files in alphabetical order anyway.Code Review
@corwinsheahan-wf @sebastianmalysa-wf @robbecker-wf
fyi @shanesizer-wf