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

Incorrect License Headers #217

Closed
YairHalberstadt opened this issue Oct 8, 2024 · 0 comments · Fixed by #218
Closed

Incorrect License Headers #217

YairHalberstadt opened this issue Oct 8, 2024 · 0 comments · Fixed by #218
Assignees
@jba
Labels
type:bug Something isn't working

Comments

@YairHalberstadt
Copy link

Description of the bug:

Many of the golang files have a header like this:

// Copyright 2024 Google LLC.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.

But no BSD style license can be found in the LICENSE file.

Either a BSD license should be added, or these headers should be corrected.

Actual vs expected behavior:

No response

Any other information you'd like to share?

No response
@YairHalberstadt YairHalberstadt added the type:bug Something isn't working label Oct 8, 2024
@jba jba self-assigned this Oct 8, 2024
jba added a commit that referenced this issue Oct 8, 2024
@jba
all: fix license headers and add a check
93947f2 
Replace all incorrect license headers with the correct one.

Add a test that checks all Go files and shell scripts.

Fixes #217.
@jba jba mentioned this issue Oct 8, 2024
Merged
jba added a commit that referenced this issue Oct 8, 2024
@jba
all: fix license headers and add a check (#218)
6d58228 
Replace all incorrect license headers with the correct one.

Add a test that checks all Go files and shell scripts.

Fixes #217.
@jba jba closed this as completed in #218 Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jba jba

Labels
type:bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

all: fix license headers and add a check google/generative-ai-go
2 participants
@jba @YairHalberstadt