save-cli icon indicating copy to clipboard operation
save-cli copied to clipboard

NPE in merging configs save.toml files

Open Cheshiriks opened this issue 3 years ago • 7 comments

When you run the fix, plugin NPE is thrown when you try to merge two save.toml files 11111111

Cheshiriks avatar Jun 28 '22 15:06 Cheshiriks

can you please provide a code snippet to reproduce this exception?

Also, SaveProperties was refactored, please check this with latest main

nulls avatar Jun 29 '22 07:06 nulls

@Cheshiriks which version did you get it with? You should post the whole log (or even attach a log file), because it contains this information in the very beginning

petertrr avatar Jun 29 '22 08:06 petertrr

@Cheshiriks which version did you get it with? You should post the whole log (or even attach a log file), because it contains this information in the very beginning

Yes. Added all log. Tested on latest version save

Cheshiriks avatar Jun 30 '22 11:06 Cheshiriks

can you please provide a code snippet to reproduce this exception?

Also, SaveProperties was refactored, please check this with latest main

Yes.

Parent save.toml

[general]
    execCmd = "java -jar ktlint --disabled_rules=standard -R diktat.jar"
    description = "22 test for diktat (linter and formatter for Kotlin)"
    language = "Kotlin"
    # this is the default value, you don't need to add it explicitly, but can be useful, if you have different pattern:
    expectedWarningsPattern = "// ;warn:?(.*):(\\d*): (.+)"
    timeOutMillis = 30000

child

[general]
    tags = ["fix"]
    description = "Test autofixing of issues discovered by diKTat"

[fix]
    execFlags = "-F"

Cheshiriks avatar Jun 30 '22 11:06 Cheshiriks

@Cheshiriks how did you run save-cli (cli args)?

nulls avatar Jul 01 '22 06:07 nulls

@nulls it's on the updated screenshot in the first post: save.exe ..\save\test --log all

petertrr avatar Jul 01 '22 11:07 petertrr

@nulls it's on the updated screenshot in the first post: save.exe ..\save\test --log all

yeah, thanks. found it!

nulls avatar Jul 01 '22 11:07 nulls