search

SERG-Delft / andy

Andy assesses student's test code. It's used in CSE1110, TU Delft.
MIT License
78 stars 23 forks source link

Add code check for if a method is invoked #228

Closed mauricioaniche closed 1 year ago

mauricioaniche commented 1 year ago

We have a code check that looks if a method was invoked inside a test. We need another one that looks for method invocations anywhere.

sandemiroren1 commented 1 year ago

Hello, me and Kiril would also like to work on this one as well.

mauricioaniche commented 1 year ago

@sandemiroren1 great! What's Kiril's github id?

mauricioaniche commented 1 year ago

In this one, make sure you reuse the code between the "method called anywhere" and the "method called in tests" which exists already.

Zakrok09 commented 1 year ago

Hello, I am Kiril!

Zakrok09 commented 1 year ago

Sir, shall we open a new branch ourselves, or should we wait for a branch that is tied to this issue? Thanks! image

mauricioaniche commented 1 year ago

@Zakrok09 You have to use Github's way of opening a PR in a OSS project. You first fork the project. And then when opening the PR, github will give you the option to open the PR in the main project!