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

azure-core 0.1.15 #7551

Merged
merged 2 commits into from
Mar 15, 2021
Merged

azure-core 0.1.15 #7551

merged 2 commits into from
Mar 15, 2021

Conversation

clearlydefinedbot
Copy link
Contributor

Type: Missing

Summary:
azure-core 0.1.15

Details:
Add MIT License

Resolution:
License Url:
https://github.com/Azure/azure-sdk-for-ruby/blob/master/LICENSE.txt

Description:

Pull request generated by Microsoft tooling.

Affected definitions:

@clearlydefinedbot
Copy link
Contributor Author

You can review the change introduced to the full definition at ClearlyDefined.

@GiriB
Copy link

GiriB commented Oct 19, 2020

The actual repository for azure-core is this which is missing LICENSE.txt. I've created a PR to add one.
However, I just saw now that the Ruby Gems page for azure-core already has a LICENSE info attached.

image

@capfei
Copy link
Member

capfei commented Oct 29, 2020

I agree with Apache. The file headers also state this:

# # Copyright (c) Microsoft and contributors. All rights reserved.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#   http://www.apache.org/licenses/LICENSE-2.0

@ariel11
Copy link
Contributor

ariel11 commented Mar 15, 2021

@capfei - I agree with Apache-2.0 here. I think we can merge this then?

@capfei capfei merged commit 8454633 into master Mar 15, 2021
@capfei capfei deleted the clearlydefinedbot_201019_061051.277 branch March 15, 2021 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants