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.

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

[METRICS-3631] Added compile phases before checks #45

Merged
merged 2 commits into from
Sep 14, 2021

Conversation

IvanVan
Copy link

@IvanVan IvanVan commented Sep 8, 2021

Description

Add compile(test-compile) phase before run checks, because semaphore cache store command caches only target folder of druid directory, not target folders of each directories.

Key changed/added classes in this PR
  • Add compile phase before run forbiddenapi check
  • Add compile-test phase before run animal sniffer check

@IvanVan IvanVan requested a review from xvrl September 8, 2021 03:57
@IvanVan IvanVan changed the title [METRICS-3631] Added phases before checks [METRICS-3631] Added compile phases before checks Sep 8, 2021
@@ -62,6 +63,7 @@ blocks:

- name: "forbidden api checks"
commands:
- ${MVN} compile ${MAVEN_SKIP} ${MAVEN_SKIP_TESTS}
Copy link
Member

Choose a reason for hiding this comment

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

why not test-compile here as well since we also check test code below?

Copy link
Author

Choose a reason for hiding this comment

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

Fixed

@IvanVan IvanVan requested a review from xvrl September 14, 2021 18:15
@IvanVan IvanVan merged commit 4b04db0 into 0.21.0-confluent Sep 14, 2021
airlock-confluentinc bot pushed a commit that referenced this pull request Sep 11, 2024
* Use camelCase

* Remove comment
pagrawal10 added a commit that referenced this pull request Sep 11, 2024
* Report broker metrics (#43)

* Update tests for these metrics (#44)

* Use camelCase (#45)

* Use camelCase

* Remove comment
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