Skip to content

Minor clean up commits#64

Merged
kenmccracken merged 4 commits intomasterfrom
clean-up
Jul 25, 2019
Merged

Minor clean up commits#64
kenmccracken merged 4 commits intomasterfrom
clean-up

Conversation

@tsteenbe
Copy link
Contributor

See individual commits for details.

tsteenbe added 3 commits July 10, 2019 22:04
Open source notices are not needed for open source projects
only when you combine OSS with proprietary code in a single binary.

Signed-off-by: Thomas Steenbergen <thomas.steenbergen@here.com>
Travis decomissioned its .org service and moved all OSS projects
over to it .com domain as such the `Build Status` link broke.

Signed-off-by: Thomas Steenbergen <thomas.steenbergen@here.com>
Signed-off-by: Thomas Steenbergen <thomas.steenbergen@here.com>
Copy link
Contributor

@kenmccracken kenmccracken left a comment

Choose a reason for hiding this comment

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

Is this a change in policy? Because I seem to remember we were asked to create this file for open source compliance.

Copy link
Contributor

@kenmccracken kenmccracken left a comment

Choose a reason for hiding this comment

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

duplicate comment

@@ -1,4 +1,4 @@
[![Build Status](https://travis-ci.org/heremaps/here-aaa-java-sdk.svg?branch=master)](https://travis-ci.org/heremaps/here-aaa-java-sdk)
[![Build Status](https://travis-ci.com/heremaps/here-aaa-java-sdk.svg?branch=master)](https://travis-ci.com/heremaps/here-aaa-java-sdk)
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you'll need to merge from master to fix the Travis CI build.

@kenmccracken kenmccracken merged commit 2886f8a into master Jul 25, 2019
@kenmccracken kenmccracken deleted the clean-up branch July 25, 2019 19:25
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.

2 participants