search

exercism / erlang-tests-generator

MIT License
1 stars 8 forks source link

WIP: Add initial dialyzer workflow file #22

Closed rlebedys closed 5 years ago

rlebedys commented 5 years ago

Add initial file to claim issue #20

NobbZ commented 5 years ago

I've added WIP: prefix to the title as a reminder that its a "work in progress" and not yet ready for review.

If you feel done edit the title and remove the "WIP:" prefix.

For questions inbetween just comment. I look at my GitHub at least twice a day.

NobbZ commented 5 years ago

@Rytis-Lebedys when do you plan to continue working on this?

I'm feeling forced to close in roughly 2 days and tag as invalid if there is no progress, to give other people the opportunity to tackle this.

rlebedys commented 5 years ago

I am working on this right now. Do you have any ideas how could I test it?

I found that I can test in on my fork. So NVM about testing. :)

rlebedys commented 5 years ago

Compilation seems to be working.

Can you elaborate about dialyzer action? What should it test and what is the expected output?

NobbZ commented 5 years ago

As I have missed to prepare the makefile for dialyzer, and currently I'm not even sure if I already use specs in the code, I think I'll prepare for that one myself and merge your workflow for now.

Thank you for the effort you put into this!