Implemented error reporting #139
Conversation
…ser info to comment.
| apply plugin: 'idea' | ||
|
|
||
| dependencies { | ||
| compile group: 'org.eclipse.mylyn.github', name: 'org.eclipse.egit.github.core', version: '2.1.5' |
There was a problem hiding this comment.
может быть следовать одному стандарту описания зависимостей?
There was a problem hiding this comment.
какому?
В features-extraction есть такое:
testCompile group: 'org.hamcrest', name: 'hamcrest-all', version: '1.3'
Там тогда тоже надо менять
There was a problem hiding this comment.
ну хотя бы в пределах одного файла
There was a problem hiding this comment.
так там остальное это jar и модули, они по другому подключаются же.
There was a problem hiding this comment.
Не понял, как ты хочешь написать.
There was a problem hiding this comment.
ну через colon
'org.eclipse.mylyn.github:org.eclipse.egit.github.core:2.1.5'
There was a problem hiding this comment.
Maven при копировании для gradle предлагает тот вариант, который сейчас.
There was a problem hiding this comment.
ну да и ладно, пусть так будет
|
|
||
| /** | ||
| * Extends the standard class to provide the hash of the thrown exception stack trace. | ||
| * @author patrick (17.06.17). |
|
|
||
| GitHubErrorBean(Throwable throwable, String lastAction) { | ||
| super(throwable, lastAction); | ||
| final int hashCode = Arrays.hashCode(throwable.getStackTrace()); |
There was a problem hiding this comment.
тут отрицательные числа вылезают, они как то странно выглядят. а можно это превратить в положительное число и вернуть стринг типа hex-value?
There was a problem hiding this comment.
Ну по идее можно привести к более большему типу и вычесть Integer.MIN
There was a problem hiding this comment.
Long.toHexString(Integer.toUnsignedLong(...))
There was a problem hiding this comment.
сейчас так: e2a83758
0x писать или так нормально?
| result = new SubmittedReportInfo(htmlUrl, message, isNewIssue ? SubmissionStatus.NEW_ISSUE : SubmissionStatus.DUPLICATE); | ||
| return result; | ||
| } catch (Exception e) { | ||
| return new SubmittedReportInfo(null, |
There was a problem hiding this comment.
давай еще приложим ссылку на github/issues типа пойди ручками отправь, мы не смогли.
| return new SubmittedReportInfo(HTML_URL_TO_CREATE_NEW_ISSUE, | ||
| ArchitectureReloadedBundle.message("report.error.connection.failure", | ||
| HTML_URL_TO_CREATE_NEW_ISSUE, | ||
| e.getMessage()), |
There was a problem hiding this comment.
тут оно не оч читаемо, лучше убрать часть про error reason
When any error occurred in plugin it will be reported as github issue.