4:05 a.m.
Hey guys,
I just noticed that both modules mentioned in the subject contain lots of
duplicated code. Since both modules are pretty similar (from the code point
of view), maybe we should collapse them into one?
Thanks,
Sebastian
6:53 a.m.
-1 We're not doing any refactoring at least not at this point. In the
future at some point we may need to revisit though.
On Mon, 8 Oct 2018 at 12:10, Sebastian Laskawiec <slaskawi(a)redhat.com>
wrote:
Hey guys,
I just noticed that both modules mentioned in the subject contain lots of
duplicated code. Since both modules are pretty similar (from the code point
of view), maybe we should collapse them into one?
Thanks,
Sebastian
_______________________________________________
keycloak-dev mailing list
keycloak-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/keycloak-dev
4:22 a.m.
Of course! Let's focus on our current priorities now.
In case we wanted to revisit this in the future, here's a JIRA:
https://issues.jboss.org/browse/KEYCLOAK-8517
On Mon, Oct 8, 2018 at 1:53 PM Stian Thorgersen <sthorger(a)redhat.com> wrote:
-1 We're not doing any refactoring at least not at this point. In
the
future at some point we may need to revisit though.
On Mon, 8 Oct 2018 at 12:10, Sebastian Laskawiec <slaskawi(a)redhat.com>
wrote:
> Hey guys,
>
> I just noticed that both modules mentioned in the subject contain lots of
> duplicated code. Since both modules are pretty similar (from the code
> point
> of view), maybe we should collapse them into one?
>
> Thanks,
> Sebastian
> _______________________________________________
> keycloak-dev mailing list
> keycloak-dev(a)lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/keycloak-dev
>
2271
days inactive
2272
days old
2 comments
2 participants
participants (2)
-
Sebastian Laskawiec -
Stian Thorgersen