java-design-patterns icon indicating copy to clipboard operation
java-design-patterns copied to clipboard

#2151 Feature/mvi pattern

Open JanFidor opened this issue 3 years ago • 4 comments

Add an MVI design pattern (includes pattern itself, tests, README.md and class diagram)

JanFidor avatar Oct 26 '22 15:10 JanFidor

The build is failing. Please investigate @JanFidor

iluwatar avatar Nov 05 '22 08:11 iluwatar

@iluwatar It seems an independent module is causing issues (page-object). My build is failing even on the master branch after syncing the fork. The first error is caused by missing test dependencies, after adding them this line throws an error java: incompatible types: java.util.List<java.lang.Object> cannot be converted to java.util.List<com.gargoylesoftware.htmlunit.html.HtmlAnchor> I'm not certain if I managed to break git on my part or if it's something else.

JanFidor avatar Nov 07 '22 16:11 JanFidor

The error from the CI build is Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.0:check (validate) on project model-view-intent: You have 2 Checkstyle violations. @JanFidor

iluwatar avatar Nov 27 '22 13:11 iluwatar

@iluwatar my bad for misinterpreting logs and sorry for the delay, university gave me plenty of work this week, but I'll get right on it now.

JanFidor avatar Dec 03 '22 13:12 JanFidor

@iluwatar fixed the violations, was a little closer to 60 after I run checkstyle locally, but it checks out now

JanFidor avatar Dec 10 '22 19:12 JanFidor

Hi, @iluwatar sorry for the long break, but the past couple of weeks have been pretty crazy on my end, but I'll be getting on it now

JanFidor avatar Jan 10 '23 13:01 JanFidor

🛠 Lift Auto-fix

Some of the Lift findings in this PR can be automatically fixed. You can download and apply these changes in your local project directory of your branch to review the suggestions before committing.[^1]

# Download the patch
curl https://lift.sonatype.com/api/patch/github.com/iluwatar/java-design-patterns/2177.diff -o lift-autofixes.diff

# Apply the patch with git
git apply lift-autofixes.diff

# Review the changes
git diff

Want it all in a single command? Open a terminal in your project's directory and copy and paste the following command:

curl https://lift.sonatype.com/api/patch/github.com/iluwatar/java-design-patterns/2177.diff | git apply

Once you're satisfied commit and push your changes in your project. [^1]: You can preview the patch by opening the patch URL in the browser.

sonatype-lift[bot] avatar Jan 17 '23 22:01 sonatype-lift[bot]

@iluwatar I made the changes you suggested, I'm not certain what to do about sonatype-lift errors, especially when it comes to Lombok decorators and deleting the final keyword which would cause chackstyle errors (just like in the FSM PR)

JanFidor avatar Jan 18 '23 15:01 JanFidor

Lift doesn't seem to handle Lombok annotations too well, so I think we can disregard them.

iluwatar avatar Jan 21 '23 09:01 iluwatar

Looks good! Thank you for contributing this new pattern 🎉

@all-contributors please add @JanFidor for code

iluwatar avatar Jan 21 '23 09:01 iluwatar

@iluwatar

I couldn't determine any contributions to add, did you specify any contributions? Please make sure to use valid contribution names.

@JanFidor already contributed before to code

allcontributors[bot] avatar Jan 21 '23 09:01 allcontributors[bot]