Skip to content

Major overhaul #20

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 25, 2017
Merged

Major overhaul #20

merged 1 commit into from
Jun 25, 2017

Conversation

larzw
Copy link
Contributor

@larzw larzw commented May 28, 2017 •
edited
Loading

  • Created standard GitHub project stucture. Included in this was an MIT License file
    with a copyright of Codecov and contributors.
  • Switched to using xUnit.
  • Cleaned up code.
  • Cleaned up readme.
  • Fixed solution name.
  • Added Codecov badge.
  • Added AppVeyor badge.
  • Fixed OpenCover filters.

@larzw larzw changed the title Major overhaul Major overhaul- Work In Progress May 28, 2017
@codecov
Copy link

codecov bot commented May 28, 2017 •
edited
Loading

Codecov Report

Merging #20 into master will decrease coverage by 19.04%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #20       +/-   ##
===========================================
- Coverage   85.71%   66.66%   -19.05%     
===========================================
  Files           2        1        -1     
  Lines          21        9       -12     
===========================================
- Hits           18        6       -12     
  Misses          3        3
Impacted Files Coverage Δ
UnitTestTargetProject/MyTargetClass.cs 66.66% <ø> (ø) ⬆️

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 23193fe...fbde3de. Read the comment docs.

@larzw
Copy link
Contributor Author

larzw commented May 28, 2017 •
edited
Loading

@stevepeak,

Your suggestions and comments especially on the README file would be appreciated. I also added a license file. Is that license ok with you? Thanks!

@larzw
Copy link
Contributor Author

larzw commented May 29, 2017

I think the OpenCover filters where wrong/missing too, which is why the coverage changed so much.

@larzw
Major overhaul
fbde3de 
* Created standard GitHub project stucture. Included in this was an MIT License file
with a copyright of Codecov and contributors.
* Switched to using xUnit.
* Cleaned up code.
* Cleaned up readme.
* Fixed solution name.
* Added Codecov badge.
* Added AppVeyor badge.
* Fixed OpenCover filters.
@larzw
Copy link
Contributor Author

larzw commented Jun 12, 2017

@stevepeak

The example C# project StyleCopAnalyzers is now using Codecov-exe. I've updated the overhauled readme with that info. Would you mind taking a look at this PR when you get a chance? Thanks!

@larzw larzw requested a review from stevepeak June 12, 2017 19:35
@larzw larzw changed the title Major overhaul- Work In Progress Major overhaul Jun 13, 2017
@stevepeak stevepeak merged commit c2f2c5e into codecov:master Jun 25, 2017
@stevepeak
Copy link
Contributor

Thank you @larzw 👍

@larzw larzw deleted the overhaul branch June 25, 2017 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stevepeak stevepeak Awaiting requested review from stevepeak

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@larzw @stevepeak