hasclass
Repos
63
Followers
40

Events

issue comment
Support `mix compile` Output as Input

Everything transferred.

Thanks for taking this package on a new journey!

Created at 1 month ago
issue comment
Support `mix compile` Output as Input

For that we need to;

  • [ ] Jason: remove JasonMiller/fix_warnings repo, so that I can
  • [ ] Seb: transfer repo to JasonMiller
  • [ ] Jason: let me know your hex.pm username, so that I can
  • [ ] seb: transfer hex package
Created at 1 month ago

add formatter and run mix format

run mix deps.update --all

update source matcher to work with current style of mix output

deprecated style does not include a function reference

use option parser with :strict option and :alias for quiet

add arg alias for "file"

update readme

add support for reading errors from mix compile output

move Util tests to UtilTest

remove debugging output

eat dog food 🐶

add a todo 😉

add more todos!

Merge pull request #1 from JasonMiller/support-mix-compile-output-as-input

Support mix compile Output as Input

Created at 1 month ago
pull request closed
Support `mix compile` Output as Input

Hello, @hasclass!

In typical developer fashion, I came across your repo after having spent a bunch of time working on my own solution. Looked like it did the trick, but it was untouched for 6 years. Figured I'd give it a go, and discovered that it didn't support the modern format of mix compile output.

What Changed?

  • formatted the repo
  • updated the deps
  • updated the Util @source_mapper to support the new format
  • added up-to-date OptionParser arguments (and some nice aliases)
  • here's the exciting one: added support for direct input from the mix compile command!
  • ate the dog food 😄

and now...

$ mix compile --force --warnings-as-errors
Compiling 7 files (.ex)
Generated fix_warnings app
$ mix test
Compiling 7 files (.ex)
Generated fix_warnings app
warning: unused alias Util
  test/fix_warnings_test.exs:5

..........

Finished in 0.04 seconds (0.00s async, 0.04s sync)
10 tests, 0 failures

Next?

I added some todos!

Created at 1 month ago
issue comment
Support `mix compile` Output as Input

Woah. Epic PR! That was a nice surprise. Thanks.

Looks excellent. As I don't have much time to maintain this, I wonder if you're interested in taking over maintainer/owner-ship? If that's the case, I'll transfer it over to you.

Created at 1 month ago