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

Fix some minor typos #24

Merged
merged 1 commit into from
Jun 17, 2019
Merged

Conversation

amoeba
Copy link
Contributor

@amoeba amoeba commented Jun 17, 2019

Found as part of my review process,

openjournals/joss-reviews#1501

and contributed for the authors.

Found as part of my review process,

openjournals/joss-reviews#1501

and contributed for the authors.
@amoeba amoeba mentioned this pull request Jun 17, 2019
1 task
@codecov-io
Copy link

codecov-io commented Jun 17, 2019

Codecov Report

Merging #24 into master will increase coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
+ Coverage   88.12%   88.25%   +0.12%     
==========================================
  Files           9        9              
  Lines        1583     1583              
  Branches      276      276              
==========================================
+ Hits         1395     1397       +2     
+ Misses        122      119       -3     
- Partials       66       67       +1
Impacted Files Coverage Δ
vixen/vixen.py 88.4% <0%> (+0.4%) ⬆️

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 a2b450f...1826420. Read the comment docs.

@prabhuramachandran
Copy link
Collaborator

Many thanks for this! LGTM, merging!

@prabhuramachandran prabhuramachandran merged commit 819366c into vixen-project:master Jun 17, 2019
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.

3 participants