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

Port: Webhook alert token and new user alerts #1338

Merged
merged 2 commits into from
Jun 18, 2024

refactor user subject

c26b1cb
Select commit
Loading
Failed to load commit list.
Merged

Port: Webhook alert token and new user alerts #1338

refactor user subject
c26b1cb
Select commit
Loading
Failed to load commit list.
Codacy Production / Codacy Static Code Analysis succeeded Jun 18, 2024 in 0s

Your pull request is up to standards!

Codacy found no issues in your code

Annotations

Check warning on line 135 in notification-publisher/src/main/java/org/dependencytrack/notification/publisher/Publisher.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

notification-publisher/src/main/java/org/dependencytrack/notification/publisher/Publisher.java#L135 

These nested if statements could be combined

Check notice on line 861 in notification-publisher/src/test/java/org/dependencytrack/notification/NotificationRouterTest.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

notification-publisher/src/test/java/org/dependencytrack/notification/NotificationRouterTest.java#L861 

Unnecessary use of fully qualified name 'Assertions.assertThat' due to existing static import 'org.assertj.core.api.Assertions.assertThat'
View more details on Codacy Production