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

EPMRPP-90342 || Email notifications. Broken links are displayed in finish launch email in case launch has status 'stopped' #2010

Merged
merged 3 commits into from
Jun 25, 2024

Conversation

APiankouski
Copy link
Contributor

…nish launch email in case launch has status 'stopped'

…nish launch email in case launch has status 'stopped'
@APiankouski APiankouski requested a review from pbortnik June 24, 2024 17:11
@APiankouski APiankouski requested a review from avarabyeu as a code owner June 24, 2024 17:11
@APiankouski APiankouski changed the title EPMRPP-90342 || Email notifications. Broken links are displayed in fi… EPMRPP-90342 || Email notifications. Broken links are displayed in finish launch email in case launch has status 'stopped' Jun 24, 2024
@Override
public OperationCompletionRS stopLaunch(Long launchId, FinishExecutionRQ finishLaunchRQ,
ReportPortalUser.ProjectDetails projectDetails, ReportPortalUser user) {
@Override

Choose a reason for hiding this comment

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

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> reported by reviewdog 🐶
Line contains a tab character.

@Override
public OperationCompletionRS stopLaunch(Long launchId, FinishExecutionRQ finishLaunchRQ,
ReportPortalUser.ProjectDetails projectDetails, ReportPortalUser user) {
@Override

Choose a reason for hiding this comment

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

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> reported by reviewdog 🐶
'method def modifier' has incorrect indentation level 8, expected level should be 2.

public OperationCompletionRS stopLaunch(Long launchId, FinishExecutionRQ finishLaunchRQ,
ReportPortalUser.ProjectDetails projectDetails, ReportPortalUser user) {
@Override
public OperationCompletionRS stopLaunch(Long launchId, FinishExecutionRQ finishLaunchRQ,

Choose a reason for hiding this comment

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

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> reported by reviewdog 🐶
Line contains a tab character.

public OperationCompletionRS stopLaunch(Long launchId, FinishExecutionRQ finishLaunchRQ,
ReportPortalUser.ProjectDetails projectDetails, ReportPortalUser user) {
@Override
public OperationCompletionRS stopLaunch(Long launchId, FinishExecutionRQ finishLaunchRQ,

Choose a reason for hiding this comment

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

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> reported by reviewdog 🐶
Abbreviation in name 'finishLaunchRQ' must contain no more than '1' consecutive capital letters.

ReportPortalUser.ProjectDetails projectDetails, ReportPortalUser user) {
@Override
public OperationCompletionRS stopLaunch(Long launchId, FinishExecutionRQ finishLaunchRQ,
ReportPortalUser.ProjectDetails projectDetails, ReportPortalUser user, String baseUrl) {

Choose a reason for hiding this comment

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

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> reported by reviewdog 🐶
Line is longer than 100 characters (found 112).

ReportPortalUser.ProjectDetails projectDetails, ReportPortalUser user) {
@Override
public OperationCompletionRS stopLaunch(Long launchId, FinishExecutionRQ finishLaunchRQ,
ReportPortalUser.ProjectDetails projectDetails, ReportPortalUser user, String baseUrl) {

Choose a reason for hiding this comment

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

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> reported by reviewdog 🐶
Line contains a tab character.

@APiankouski APiankouski merged commit 3e5ca28 into develop Jun 25, 2024
7 checks passed
@APiankouski APiankouski deleted the EPMRPP-90342 branch June 25, 2024 09:00
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