add fudge-usage-stat.pl to show usage of fudge implementations, verbs…#228
Open
ronaldxs wants to merge 3 commits intoRaku:masterfrom
ronaldxs:fudge-usage-stat
Open
add fudge-usage-stat.pl to show usage of fudge implementations, verbs…#228ronaldxs wants to merge 3 commits intoRaku:masterfrom ronaldxs:fudge-usage-stat
ronaldxs wants to merge 3 commits intoRaku:masterfrom
ronaldxs:fudge-usage-stat
Conversation
JJ
approved these changes
May 6, 2020
Contributor
JJ
left a comment
JJ
left a comment
There was a problem hiding this comment.
This is an additional file that can be useful, and it does not disturb anything else. I think we can safely merge it.
vrurg
requested changes
May 6, 2020
Contributor
vrurg
left a comment
vrurg
left a comment
There was a problem hiding this comment.
There is tools/ dir in roast where scripts are supposed to end up with.
Also, I think a README briefly explaining what each tool does would be helpful or we risk ending up with loads of them without any way to quickly find the needed one.
| die <<"USAGE"; | ||
| Usage: $0 [roast directory] | ||
|
|
||
| Intended to run against directory with synopses but not many |
Contributor
There was a problem hiding this comment.
At some point we'll drift away from binding to synopses. Just 'tests' would be good enough.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…, versions etc.
As described in my comment on Issue #226, here is my script for collecting stats on usage of fudge directives. I think it would be better for roast to have a bin/ directory and for this, and later other, scripts to go there. While I won't take the initiative on adding a bin/ directory I will certainly move this script there if asked.