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

Ny EditorConfig + reformatering av code. #2500

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

Ny EditorConfig + reformatering av code. #2500

wants to merge 6 commits into from

Conversation

mrsladek
Copy link
Contributor

No description provided.

@mrsladek mrsladek requested a review from a team as a code owner June 18, 2024 15:39
Copy link

@sirimykland sirimykland left a comment

Choose a reason for hiding this comment

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

det kan se ut som at det fortsatt er noen filer som ikke er formatert
Screenshot 2024-06-19 at 09 27 12

Choose a reason for hiding this comment

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

Vi bør legge til ij_java_record_components_wrap = split_into_lines så får vi props på forskjellig linje når det er mange av dem

Screenshot 2024-06-19 at 09 41 33

Hvis vi også legger til ij_java_parameter_annotation_wrap = split_into_lines gjøre også bruken av records og klasser med mange parameter mer oversiktlig:
Screenshot 2024-06-19 at 09 49 24

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Lagt inn i iterasjon 2.

@@ -112,7 +111,8 @@ void handleMessage(JournalfoeringHendelseRecord payload) {
// De uten kanalreferanse er "klonet" av SBH og journalført fra Gosys.
// Normalt blir de journalført, men det feiler av og til pga tilgang.
// Håndterer disse journalpostene senere i tilfelle SBH skal ha klart å ordne ting selv
var delay = eksternReferanseId == null && !mottaksKanal.equals(MottakKanal.SELVBETJENING.getKode()) ? Duration.ofHours(journalføringDelay) : Duration.ZERO;
var delay = eksternReferanseId == null
Copy link
Contributor

Choose a reason for hiding this comment

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

Veldig lang indent etter newline her? Er det 8 char - liker 4 char bedre

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tror den bare stikker til det forrige - tar jeg en second reformat så ser det slikt ut
image

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.

3 participants