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

Update views.py #5575

Merged
merged 1 commit into from
Dec 2, 2019
Merged

Update views.py #5575

merged 1 commit into from
Dec 2, 2019

Conversation

Solexplorer
Copy link
Contributor

@Solexplorer Solexplorer commented Dec 2, 2019

Description

@alexvotofuture changed his username in Github but the change wasn't reflected in gitcoin.co/about#team . I only updated the record there

Refers/Fixes
Testing

ezgif com-video-to-gif (1)

Alex did change his username in Github but the change wasn't reflected in gitcoin.co/about#team
@codecov
Copy link

codecov bot commented Dec 2, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5575      +/-   ##
==========================================
+ Coverage   30.25%   30.27%   +0.02%     
==========================================
  Files         247      247              
  Lines       21074    21074              
  Branches     3048     3048              
==========================================
+ Hits         6375     6381       +6     
+ Misses      14423    14417       -6     
  Partials      276      276
Impacted Files Coverage Δ
app/retail/views.py 27.67% <ø> (ø) ⬆️
app/dashboard/embed.py 31.6% <0%> (+3.44%) ⬆️

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 9cfd8e1...5a80eb7. Read the comment docs.

@thelostone-mc
Copy link
Member

Yup I would not have seen this :P

@thelostone-mc thelostone-mc merged commit a04d56c into gitcoinco:master Dec 2, 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.

2 participants