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

Communication: Accept messaging code of conduct #7154

Merged
merged 102 commits into from
Oct 14, 2023

Merge branch 'develop' into 7085/v2/accept-messaging-code-of-conduct

705d9ea
Select commit
Loading
Failed to load commit list.
Merged

Communication: Accept messaging code of conduct #7154

Merge branch 'develop' into 7085/v2/accept-messaging-code-of-conduct
705d9ea
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
Teamscale / teamscale-findings required action Oct 13, 2023

Teamscale

This pull request would introduce 11 new findings

Details

Teamscale's logo 11 added, 7 in changed code, 1 removed 74 Test Gaps (100%) tested changes: 0|untested changes: 35|untested additions: 39

  • src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java#457: Method responsibleUsers_shouldReturnCorrectValue violates naming convention. Should be one of [a-z][a-zA-Z0-9]* (view in Teamscale)
  • src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java#17: Star import of de.tum.in.www1.artemis.domain.* should not be used (view in Teamscale)
  • src/main/java/de/tum/in/www1/artemis/web/rest/metis/conversation/ConversationResource.java#3: Star import of java.util.* should not be used (view in Teamscale)
  • src/main/java/de/tum/in/www1/artemis/domain/ConductAgreement.java#5: Star import of javax.persistence.* should not be used (view in Teamscale)
  • src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java#437: Method codeOfConduct_isAccepted violates naming convention. Should be one of [a-z][a-zA-Z0-9]* (view in Teamscale)
  • src/main/java/de/tum/in/www1/artemis/domain/ConductAgreementId.java#24: Clone with 2 instances of length 13 (view in Teamscale)
  • src/main/java/de/tum/in/www1/artemis/domain/ConductAgreement.java#28: Clone with 2 instances of length 13 (view in Teamscale)
  • src/main/java/de/tum/in/www1/artemis/service/messaging/InstanceMessageReceiveService.java#293: Interface comment missing (view in Teamscale)
  • src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java#444: Method codeOfConduct_accept violates naming convention. Should be one of [a-z][a-zA-Z0-9]* (view in Teamscale)
  • src/main/java/de/tum/in/www1/artemis/service/programming/ProgrammingExerciseService.java#69: Comment should not contain // (view in Teamscale)
  • src/main/java/de/tum/in/www1/artemis/service/programming/ProgrammingExerciseService.java#62: Comment should not contain // (view in Teamscale)

Annotations

Check warning on line 437 in src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java

See this annotation in the file changed.

@teamscale teamscale / teamscale-findings

src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java#L437

Method `codeOfConduct_isAccepted` violates naming convention. Should be one of `[a-z][a-zA-Z0-9]*`
https://teamscale.io/findings.html#details/GitHub-ls1intum-Artemis?t=7085%2Fv2%2Faccept-messaging-code-of-conduct%3AHEAD&id=120B355BDC579EAD7834B4502A27C35C

Check warning on line 17 in src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java

See this annotation in the file changed.

@teamscale teamscale / teamscale-findings

src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java#L17

Star import of `de.tum.in.www1.artemis.domain.*` should not be used
https://teamscale.io/findings.html#details/GitHub-ls1intum-Artemis?t=7085%2Fv2%2Faccept-messaging-code-of-conduct%3AHEAD&id=549F645D137AB082341769FD2B6DB532

Check warning on line 444 in src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java

See this annotation in the file changed.

@teamscale teamscale / teamscale-findings

src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java#L444

Method `codeOfConduct_accept` violates naming convention. Should be one of `[a-z][a-zA-Z0-9]*`
https://teamscale.io/findings.html#details/GitHub-ls1intum-Artemis?t=7085%2Fv2%2Faccept-messaging-code-of-conduct%3AHEAD&id=C3258F9BF36325330B2EA7324A980C43

Check warning on line 457 in src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java

See this annotation in the file changed.

@teamscale teamscale / teamscale-findings

src/test/java/de/tum/in/www1/artemis/metis/ConversationIntegrationTest.java#L457

Method `responsibleUsers_shouldReturnCorrectValue` violates naming convention. Should be one of `[a-z][a-zA-Z0-9]*`
https://teamscale.io/findings.html#details/GitHub-ls1intum-Artemis?t=7085%2Fv2%2Faccept-messaging-code-of-conduct%3AHEAD&id=EE630CDC455FFB923061524636E28924

Check warning on line 3 in src/main/java/de/tum/in/www1/artemis/web/rest/metis/conversation/ConversationResource.java

See this annotation in the file changed.

@teamscale teamscale / teamscale-findings

src/main/java/de/tum/in/www1/artemis/web/rest/metis/conversation/ConversationResource.java#L3

Star import of `java.util.*` should not be used
https://teamscale.io/findings.html#details/GitHub-ls1intum-Artemis?t=7085%2Fv2%2Faccept-messaging-code-of-conduct%3AHEAD&id=7AB748D3663E46FE52A1E7C9E5BB6F30

Check warning on line 5 in src/main/java/de/tum/in/www1/artemis/domain/ConductAgreement.java

See this annotation in the file changed.

@teamscale teamscale / teamscale-findings

src/main/java/de/tum/in/www1/artemis/domain/ConductAgreement.java#L5

Star import of `javax.persistence.*` should not be used
https://teamscale.io/findings.html#details/GitHub-ls1intum-Artemis?t=7085%2Fv2%2Faccept-messaging-code-of-conduct%3AHEAD&id=73CBF151DD61434070A871688F7E4E3A

Check warning on line 51 in src/main/java/de/tum/in/www1/artemis/domain/ConductAgreement.java

See this annotation in the file changed.

@teamscale teamscale / teamscale-findings

src/main/java/de/tum/in/www1/artemis/domain/ConductAgreement.java#L28-L51

Clone with 2 instances of length 13
https://teamscale.io/findings.html#details/GitHub-ls1intum-Artemis?t=7085%2Fv2%2Faccept-messaging-code-of-conduct%3AHEAD&id=DB384D0BEA5385E7067C11DBAAE05618

Check warning on line 47 in src/main/java/de/tum/in/www1/artemis/domain/ConductAgreementId.java

See this annotation in the file changed.

@teamscale teamscale / teamscale-findings

src/main/java/de/tum/in/www1/artemis/domain/ConductAgreementId.java#L24-L47

Clone with 2 instances of length 13
https://teamscale.io/findings.html#details/GitHub-ls1intum-Artemis?t=7085%2Fv2%2Faccept-messaging-code-of-conduct%3AHEAD&id=BBE2B719848B9BE214787456E237AF76