Skip to content

Add Apache license header to spans.rs#1594

Merged
alamb merged 1 commit into
apache:mainfrom
alamb:alamb/span-rat
Dec 13, 2024
Merged

Add Apache license header to spans.rs#1594
alamb merged 1 commit into
apache:mainfrom
alamb:alamb/span-rat

Conversation

@alamb

@alamb alamb commented Dec 12, 2024

Copy link
Copy Markdown
Contributor

While trying to make a release candidate, the license check failed

Running rat license checker on /Users/andrewlamb/Software/sqlparser-rs/dev/dist/apache-datafusion-sqlparser-rs-0.53.0-rc1/apache-datafusion-sqlparser-rs-0.53.0.tar.gz
NOT APPROVED: src/ast/spans.rs (apache-datafusion-sqlparser-rs-0.53.0/src/ast/spans.rs): false

This PR adds the license to spans.rs (a newly added file)

@alamb alamb marked this pull request as ready for review December 12, 2024 14:27
@alamb

alamb commented Dec 12, 2024

Copy link
Copy Markdown
Contributor Author

I plan to cherry-pick this commit to a release branch so it doesn't block the release

@iffyio iffyio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@alamb alamb merged commit 885aa93 into apache:main Dec 13, 2024
@alamb

alamb commented Dec 13, 2024

Copy link
Copy Markdown
Contributor Author

Thanks @iffyio

@alamb alamb deleted the alamb/span-rat branch December 13, 2024 18:02
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