Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Nielswps
Copy link
Contributor

@Nielswps Nielswps commented May 8, 2020

This adds error handling for the queries and the UPPAAL connectivity

@Nielswps Nielswps added the enhancement 💡 new feature idea label May 8, 2020
@Nielswps Nielswps self-assigned this May 8, 2020
@yrke yrke marked this pull request as draft May 19, 2020 09:23
@Nielswps Nielswps changed the title WIP: Additional error handling Additional error handling May 19, 2020
@Nielswps Nielswps marked this pull request as ready for review May 19, 2020 17:55
Copy link
Member

@yrke yrke left a comment

Choose a reason for hiding this comment

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

I think it looks ok, I just have to small comments.

}
}

System.out.println(sb.toString());
Copy link
Member

Choose a reason for hiding this comment

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

Seems to be debug output?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I forgot to remove the println call, but the above is needed to wrap the text for the tooltip (i couldn't find a better built-in solution)

I have removed the println

@yrke yrke merged commit 4dc8e20 into DEIS-Tools:master May 25, 2020
@Nielswps Nielswps deleted the errorHandling branch May 25, 2020 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement 💡 new feature idea

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants