Creating IDE Specific configuration with gradle

[aepfli]

I added a small build.gradle file, which uses gradle dsl, as i found it
easiert to manipulate xml with. This snippet will add the issue link to
idea on each sync.

The questions to answer is, if we want to stick to groovy and mix kotlin
and groovy dsl. Which might be a bad idea. Sadly i have not had the time
to check if there is a convenient way to do so.

I just want to show this, and cross check if that might be suitable for
our cause or not. (i think at least for now it is)

closes: #229

---

I hereby agree to the terms of the JUnit Pioneer Contributor License Agreement.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

No Coverage information
No Duplication information

[Bukama]

First let me point out, that I'm a bit confused. I understand Marc and Vladimir that they suggest, that this should not be part of gradle, but of IDEA - but maybe I misunderstood them or forgot something we have talked about.

Aside from that:

Looks nice and works perfect! But we should mentioned that it needs a restart of IDEA, if no .idea folder was there before. I had cloned your repository, imported it to IDEA and the links inside the history didn't work. After a restart they worked smoothly.

This brings up the discussion about putting the Issue or PR number back to commit header. I'll open an bishue for that, referencing this PR and the underlying bishue.

[Bukama]

Wah, whole team review was requested, I did one and did that for the whole team :eek:

[aepfli]

Looks nice and works perfect! But we should mentioned that it needs a restart of IDEA, if no .idea folder was there before. I had cloned your repository, imported it to IDEA and the links inside the history didn't work. After a restart they worked smoothly.

It should normally work as soon as you sync your project. as the internal build.gradle is executed in this step. if you want to try again, you could go to your IDEA settings and remove the Issue Navigation-entries and retrigger a sync

[aepfli]

This brings up the discussion about putting the Issue or PR number back to commit header. I'll open an bishue for that, referencing this PR and the underlying bishue.

but there will be also clickable links in the commit message, just not as convinient as in the title

[Bukama]

This brings up the discussion about putting the Issue or PR number back to commit header. I'll open an bishue for that, referencing this PR and the underlying bishue.

but there will be also clickable links in the commit message, just not as convinient as in the title

Ah okay, didn't understand it that way 👍

Please write this down in #264

[aepfli]

to be fully annoying we could also add

  <component name="CommitMessageInspectionProfile">
    <profile version="1.0">
      <inspection_tool class="BodyLimit" enabled="true" level="WARNING" enabled_by_default="true" />
      <inspection_tool class="SubjectBodySeparation" enabled="true" level="WARNING" enabled_by_default="true" />
      <inspection_tool class="SubjectLimit" enabled="true" level="WARNING" enabled_by_default="true" />
    </profile>
  </component>

to the configuration, which will help when writing commit messages with idea. Like limit the body, adding a separation and a subjectlimit

[Bukama]

to be fully annoying we could also add

  <component name="CommitMessageInspectionProfile">
    <profile version="1.0">
      <inspection_tool class="BodyLimit" enabled="true" level="WARNING" enabled_by_default="true" />
      <inspection_tool class="SubjectBodySeparation" enabled="true" level="WARNING" enabled_by_default="true" />
      <inspection_tool class="SubjectLimit" enabled="true" level="WARNING" enabled_by_default="true" />
    </profile>
  </component>

to the configuration, which will help when writing commit messages with idea. Like limit the body, adding a separation and a subjectlimit

Can you describe this in more detail? I don't understand what you mean by "limit the body"? Why should there a limit how long the body size can be in vertical terms?

[aepfli]

BodyLimit = line length defaults to 72
subjectLimit line length defautls to 72
SubjectBodySeperation = an empty line has to be between subject and body

Those will be shown as warnings in the intellij commit dialog

[Bukama]

@aepfli If you turn this into a PR I would approved.