Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve --report=terms command #440

Merged
merged 3 commits into from
Jun 10, 2024
Merged

Conversation

ydah
Copy link
Collaborator

@ydah ydah commented Jun 9, 2024

This PR improve --report=terms command.

Output Unused Terms only when there are unused terms.
Also, output the number of terms as follows:

Before:

 Unused Terms
     0 YYerror
     2 YYUNDEF
     3 '\\\\'
     3 '\\13'
     4 keyword_class2
     5 tNUMBER
     6 tPLUS
     7 tMINUS
     8 tEQ
     9 tEQEQ
    10 '>'

After:

 11 Unused Terms
     0 YYerror
     1 YYUNDEF
     2 '\\\\'
     3 '\\13'
     4 keyword_class2
     5 tNUMBER
     6 tPLUS
     7 tMINUS
     8 tEQ
     9 tEQEQ
    10 '>'

ydah added 3 commits June 10, 2024 01:08
Before:
```
 Unused Terms
     0 YYerror
     2 YYUNDEF
     3 '\\\\'
     3 '\\13'
     4 keyword_class2
     5 tNUMBER
     6 tPLUS
     7 tMINUS
     8 tEQ
     9 tEQEQ
    10 '>'
```

After:
```
 11 Unused Terms
     0 YYerror
     1 YYUNDEF
     2 '\\\\'
     3 '\\13'
     4 keyword_class2
     5 tNUMBER
     6 tPLUS
     7 tMINUS
     8 tEQ
     9 tEQEQ
    10 '>'
```
@ydah ydah force-pushed the improve-report-unused-terms branch from 7c06a88 to fec1f15 Compare June 9, 2024 16:10
@ydah ydah merged commit 13140bc into ruby:master Jun 10, 2024
16 checks passed
@ydah ydah deleted the improve-report-unused-terms branch June 10, 2024 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant