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

Add academic bibliographic references in the README #152

Merged
merged 2 commits into from
Oct 20, 2020
Merged

Conversation

fermadeiral
Copy link
Collaborator

This PR adds academic bibliographic references in the README. I realized that this information was missing in our repository because of @slarse's comment here.

Currently, there is only @HarisAdzemovic's thesis, but soon we will have @henry-lp's thesis and other related scientific works.

@monperrus, WDYT?

@codecov
Copy link

codecov bot commented Oct 16, 2020

Codecov Report

Merging #152 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #152   +/-   ##
=========================================
  Coverage     82.49%   82.49%           
  Complexity      350      350           
=========================================
  Files            36       36           
  Lines          1365     1365           
  Branches        198      198           
=========================================
  Hits           1126     1126           
  Misses          144      144           
  Partials         95       95           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ea7d626...d4cb07c. Read the comment docs.

@slarse
Copy link
Collaborator

slarse commented Oct 16, 2020

I was actually about to suggest this at the same time as adding that comment, but then I didn't for some reason... As a visitor to the repo, I'd definitely be more impressed with some academic references in the README (and let's face it, the top level README is to roughly 95% an advert for the project).

@monperrus
Copy link
Contributor

Great idea thanks @fermadeiral !

we can replace the full bibtex by a link [bibtex] pointing to the bibtex export of Diva.

@monperrus monperrus merged commit e7bb394 into master Oct 20, 2020
@monperrus monperrus deleted the update-readme branch October 20, 2020 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants