-
Notifications
You must be signed in to change notification settings - Fork 620
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
Org: optimize #3611
Org: optimize #3611
Conversation
However,... one tag is lost with this change. I inspected this. The missing one should not be tagged. |
Codecov ReportBase: 82.84% // Head: 82.84% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #3611 +/- ##
=======================================
Coverage 82.84% 82.84%
=======================================
Files 223 223
Lines 54280 54279 -1
=======================================
Hits 44966 44966
+ Misses 9314 9313 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
ff1610b
to
3804034
Compare
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
3804034
to
70de9e4
Compare
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
…es starting from "#" earlyer Signed-off-by: Masatake YAMATO <yamato@redhat.com>
70de9e4
to
d068172
Compare
I found the Org parser extracts many noises. It may be rewritten more line-oriented way. |
The Org parser is incredibly slow.
The original:
With this pull request:
It becomes 150% faster with this pull request.
BTW, this parser doesn't fill scope fields.