Skip to content

dependency check with inherit instead of aggregate#11709

Merged
clintropolis merged 1 commit intoapache:masterfrom
clintropolis:depedency-check-no-aggregate
Sep 15, 2021
Merged

dependency check with inherit instead of aggregate#11709
clintropolis merged 1 commit intoapache:masterfrom
clintropolis:depedency-check-no-aggregate

Conversation

@clintropolis
Copy link
Copy Markdown
Member

I'm having some trouble building release artifacts due to the dependency check failing because it tries to find CVE in druid-integration-tests (which is unnecessary). This is just undoing the change to use aggregate from #10883 and switching back to using inheritance so that benchmarks/distribution/integration-tests are all skipped.

Hopefully the cron job isn't too flaky this way...

@clintropolis clintropolis added Area - Dependencies Area - Dev For items related to the project itself, like dev docs and checklists, but not CI labels Sep 14, 2021
Copy link
Copy Markdown
Contributor

@suneet-s suneet-s left a comment

Choose a reason for hiding this comment

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

LGTM

@clintropolis clintropolis changed the title dependency check with inhert instead of aggregate dependency check with inherit instead of aggregate Sep 15, 2021
@clintropolis clintropolis merged commit 1370fcf into apache:master Sep 15, 2021
clintropolis added a commit to clintropolis/druid that referenced this pull request Sep 15, 2021
@clintropolis clintropolis deleted the depedency-check-no-aggregate branch September 15, 2021 11:19
@clintropolis clintropolis added this to the 0.22.0 milestone Sep 15, 2021
@jihoonson jihoonson mentioned this pull request Apr 18, 2022
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area - Dependencies Area - Dev For items related to the project itself, like dev docs and checklists, but not CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants