Skip to content

[fix][ci] Check full build artifacts licenses - #18171

Merged
Technoboy- merged 2 commits into
apache:masterfrom
tisonkun:check-sql-distro-licenses
Oct 24, 2022
Merged

[fix][ci] Check full build artifacts licenses#18171
Technoboy- merged 2 commits into
apache:masterfrom
tisonkun:check-sql-distro-licenses

Conversation

@tisonkun

@tisonkun tisonkun commented Oct 24, 2022

Copy link
Copy Markdown
Member

This closes #16783.
This closes #15253.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository: tisonkun#8

This closes apache#16783.
This closes apache#15253.

Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
@tisonkun

Copy link
Copy Markdown
Member Author

/pulsarbot run-failure-checks

Comment thread .github/workflows/pulsar-ci.yaml
@tisonkun

Copy link
Copy Markdown
Member Author

"Pulsar CI / Build Pulsar docker image" passed. I'm gonna rerun other tests later :(

@tisonkun

Copy link
Copy Markdown
Member Author

/pulsarbot run-failure-checks

@Technoboy- Technoboy- added this to the 2.12.0 milestone Oct 24, 2022
@Technoboy-
Technoboy- merged commit e913a10 into apache:master Oct 24, 2022
@tisonkun
tisonkun deleted the check-sql-distro-licenses branch October 24, 2022 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci doc-not-needed Your PR changes do not impact docs ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

check-binary-license.sh doesn't check Pulsar SQL licenses in CI

4 participants