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

added information on contribution #48

Merged
merged 4 commits into from
May 14, 2020
Merged

added information on contribution #48

merged 4 commits into from
May 14, 2020

Conversation

tuxette
Copy link

@tuxette tuxette commented May 5, 2020

As you suggested, I added our names and emails in the contributor list (in DESCRIPTION: I did not find a CITATION file). Thanks!
I also removed the codecov link (that is not working for some reasons that I don't quite understand: I'll check later).
Do not hesitate to tell us in which direction to work. I think that we have the package and scripts well in hand now.
Regards,
Nathalie

Copy link
Contributor

@seabbs seabbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great thanks for this. Will follow up with an email shortly (as you may have seen in the issues we have had a memory leak that has been taking up all my time resolving)

@seabbs seabbs merged commit 6d0ed94 into epiforecasts:master May 14, 2020
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.

2 participants