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

kbs: report numbers update #15

Merged
merged 1 commit into from
Dec 14, 2016
Merged

Conversation

michamos
Copy link
Contributor

  • Add new often cited report numbers, found with
    invenio-scripts/unrecognized_report_numbers.py

  • Relax patterns to allow dropping leading 0

  • Replace all ' ' patterns by 's' to allow additional spaces

  • Cosmetic improvement: align second column

Signed-off-by: Micha Moskovic michamos@gmail.com

* Add new often cited report numbers, found with
invenio-scripts/unrecognized_report_numbers.py

* Remove broken pattern escaping

* Relax patterns to allow dropping leading 0

* Replace all ' ' patterns by 's' to allow additional spaces

* Cosmetic improvement: align second column

Signed-off-by: Micha Moskovic <michamos@gmail.com>
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 77.503% when pulling 95b59b8 on michamos:report-nums into c697194 on inspirehep:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 77.503% when pulling 95b59b8 on michamos:report-nums into c697194 on inspirehep:master.

@michamos
Copy link
Contributor Author

@kaplun could you please have a look at this?

@kaplun
Copy link
Contributor

kaplun commented Dec 14, 2016

@michamos LGTM.

@kaplun kaplun merged commit 47710a2 into inspirehep:master Dec 14, 2016
michamos added a commit to michamos/invenio that referenced this pull request Apr 11, 2017
* Backports inspirehep/refextract#15.
* Adds new often cited report numbers, found with
invenio-scripts/unrecognized_report_numbers.py.
* Removes broken pattern escaping.
* Relaxes patterns to allow dropping leading 0.
* Replaces all ' ' patterns by 's' to allow additional spaces.
* Cosmetic improvement: aligns second column.

Signed-off-by: Micha Moskovic <michamos@gmail.com>
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.

None yet

3 participants