Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Mar 05 20:26

    Fabrice-TIERCELIN on master

    [cleanup] erefactor/AutoRefacto… [format] erefactor - format sou… (compare)

  • Mar 05 20:26
    Fabrice-TIERCELIN closed #475
  • Mar 04 14:49
    cal101 synchronize #475
  • Mar 04 07:55

    Fabrice-TIERCELIN on master

    Remove tab indentations (compare)

  • Mar 04 06:47
    Fabrice-TIERCELIN closed #477
  • Mar 04 06:47

    Fabrice-TIERCELIN on master

    [cleanup] erefactor/EclipseJdt … [format] erefactor - format sou… (compare)

  • Mar 04 06:46

    Fabrice-TIERCELIN on master

    [cleanup] erefactor/EclipseJdt … (compare)

  • Mar 04 06:46
    Fabrice-TIERCELIN closed #476
  • Mar 04 06:45

    Fabrice-TIERCELIN on master

    [cleanup] erefactor/AutoRefacto… (compare)

  • Mar 04 06:45
    Fabrice-TIERCELIN closed #474
  • Mar 03 21:14
    cal101 opened #477
  • Mar 03 21:11
    cal101 opened #476
  • Mar 03 20:43
    cal101 commented #474
  • Mar 03 20:37
    cal101 synchronize #474
  • Mar 03 20:34
    cal101 synchronize #474
  • Mar 03 19:07
    cal101 commented #474
  • Mar 03 18:30
    Fabrice-TIERCELIN commented #474
  • Mar 03 18:26

    Fabrice-TIERCELIN on master

    [cleanup] erefactor/EclipseJdt … [format] erefactor - format sou… (compare)

  • Mar 03 18:26
    Fabrice-TIERCELIN closed #473
  • Mar 03 18:25

    Fabrice-TIERCELIN on master

    [format] format source code [cleanup] erefactor/EclipseJdt … [format] erefactor - format sou… (compare)

Fabrice TIERCELIN
@Fabrice-TIERCELIN
Assign me #172
Luis Cruz
@luiscruz
Hi Jean-Noël!
Can you please review #213 ?
Jean-Noël Rouvignac
@JnRouvignac
Hi Luis
PR 213?
Luis Cruz
@luiscruz
yap
Jean-Noël Rouvignac
@JnRouvignac
OK, I can now see you've addressed the review comments
Yep I'll try to
Luis Cruz
@luiscruz
In order to fix PR #212 I need the method getAncestorOrNull that I’ve created on that one.
Jean-Noël Rouvignac
@JnRouvignac
it starts to be hard for me to keep up between yours PRs and Fabrice's :)
ok
Luis Cruz
@luiscruz
yea, I understand ;)
Jean-Noël Rouvignac
@JnRouvignac
OK there is a lot in this one
The rewrites are currently a bit blurred
it is hard to read when comes this level of rewriting
but I think I'll clean that up
I made several new comments
Please ping me once you'll have acted on them and pushed the new code for review
Luis Cruz
@luiscruz
ok
Luis Cruz
@luiscruz
ping :)
Luis Cruz
@luiscruz
I’ve fixed your code reviews. Let me know what is the next step
Jean-Noël Rouvignac
@JnRouvignac
Next step is tomorrow
:)
Luis Cruz
@luiscruz
deal! :D
Fabrice TIERCELIN
@Fabrice-TIERCELIN
Should I take #222?
Jean-Noël Rouvignac
@JnRouvignac
@Fabrice-TIERCELIN please do if you feel confident
be careful there are many corner cases here
methods (you can ignore parameters), fields, variables can be shadowed
etc.
This is pretty tough
maybe start small and provide a simple fix
then we'll take it forward from there
Jean-Noël Rouvignac
@JnRouvignac
@luiscruz I have looked at the PR and I have several problems with it
first of all mvn clean install fails
for the rest, maybe this is not the best medium here to explain them
last, reviewing PRs on github is not great IMO :(
but anyway we'll get through this
Jean-Noël Rouvignac
@JnRouvignac
What I mean is that there is more work needed before I can merge it
Luis Cruz
@luiscruz
Hi
I’ll see that now
Jean-Noël Rouvignac
@JnRouvignac
hello
I'll speak to you on hangout to avoid polluting this channel with remarks towards the PR
if you log in
:)
Luis Cruz
@luiscruz
are we talking about JnRouvignac/AutoRefactor#226?
Jean-Noël Rouvignac
@JnRouvignac
yes
Luis Cruz
@luiscruz
alright
we can speak on hangout
when are you available?
Jean-Noël Rouvignac
@JnRouvignac
let's chat now :)