next_rails icon indicating copy to clipboard operation
next_rails copied to clipboard

Adds support to print info for multiple files.

Open mateusdeap opened this issue 3 years ago • 3 comments

Description

This is a bit of a 2 for 1:

  1. It adds a new option, --path, so that we can provide a custom path to our deprecation files.
  2. With this option we can also give the deprecations command a file glob. That way it will print out information on the deprecation warnings present in all the files that match that glob.

And that's basically it.

Motivation and Context

The motivation is given on the issue it fixes. Fixes #3.

How Has This Been Tested?

Manually on the command line since the info command is entirely contained in the exe/deprecations script.

I will abide by the code of conduct

mateusdeap avatar Feb 10 '23 19:02 mateusdeap

@mateusdeap While I think issue #3 was asking for a way to merge multiple shitlist json files, I think this is a good step in that direction. Since we have the deprecations from all the json files in one place now, we can certainly create a new merge mode which will create a new json file with results merged from shitlist files.

Let me know if I am understanding this all a bit differently.

rishijain avatar Feb 14 '23 12:02 rishijain

@etagwerker @rishijain Sorry for the late response. I'll later try to address all of these comments

mateusdeap avatar Jun 12 '23 12:06 mateusdeap

@mateusdeap Sounds good, thank you!

etagwerker avatar Jun 15 '23 01:06 etagwerker