Skip to content

Conversation

Taz03
Copy link
Member

@Taz03 Taz03 commented Dec 14, 2022

image

it points to a class which is not available

@Taz03 Taz03 added bug Something isn't working priority: normal labels Dec 14, 2022
@Taz03 Taz03 requested review from a team as code owners December 14, 2022 17:18
@Taz03 Taz03 self-assigned this Dec 14, 2022
Zabuzard
Zabuzard previously approved these changes Dec 15, 2022

@Override
public void onChannelCreate(@Nonnull ChannelCreateEvent createEvent) {
public void onChannelCreate(ChannelCreateEvent createEvent) {
Copy link
Member

Choose a reason for hiding this comment

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

can u double check other EventReceivers in the code base? Im pretty sure I copied this method from somewhere in the code base. So maybe that other user also has @Nonnull.

Copy link
Member Author

Choose a reason for hiding this comment

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

ohh yea there was more

@Taz03 Taz03 changed the title Fixed package info for help package Fixed package info for help package and removed NotNull annotations Dec 15, 2022
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@Taz03 Taz03 requested a review from Zabuzard December 15, 2022 13:56
@Zabuzard
Copy link
Member

merging today, so that it makes it into the last release for this year, tomorrow :)

@Zabuzard Zabuzard merged commit 2524271 into develop Dec 16, 2022
@Zabuzard Zabuzard deleted the bugfix/package-info branch December 16, 2022 07:36
@Zabuzard Zabuzard mentioned this pull request Dec 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority: normal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants