Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

Heimdall 1 #22

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Heimdall 1 #22

wants to merge 4 commits into from

Conversation

aakash17kc
Copy link
Contributor

@aakash17kc aakash17kc commented Jun 28, 2020

Added email notification.
Removed parent pom.
Removed Jenkins url from code(was hardcoded).
Renamed packages.
Gson was replaced with Jackson.
Updated Readme for new code.
Updated properties file with email properties

<dependency>
<groupId>com.fasterxml.jackson.core</groupId>
<artifactId>jackson-core</artifactId>
<version>2.11.1</version>
Copy link
Collaborator

Choose a reason for hiding this comment

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

version to be added in properties tag

@aakash17kc aakash17kc requested a review from vaibsaraswat June 29, 2020 06:51
reqEntity.addTextBody("channel", slackChannelInfo.channelName).addTextBody("as_user", "true")
.addTextBody("text", slackMessageInfo.testExecutionInfo.toString());
final String generatedFileName = Files.write(Paths.get(ATTACHMENT_FILE_NAME),
buildResultDto.failedTestDescription.toString().getBytes()).normalize().toAbsolutePath()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll update the code for failedtestcount == 0

@ntantri
Copy link
Collaborator

ntantri commented Jul 16, 2020

Can you resolve the conflicts?

Copy link
Collaborator

@ntantri ntantri left a comment

Choose a reason for hiding this comment

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

Good to go

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants