Skip to content

Conversation

@MunifTanjim
Copy link
Contributor

@MunifTanjim MunifTanjim commented Jul 12, 2022

Ref: #418

@MunifTanjim MunifTanjim changed the base branch from v2.x to main July 12, 2022 09:13
@MunifTanjim
Copy link
Contributor Author

@cseickel can you please login to https://codecov.io and allow it to access this repository?

@cseickel
Copy link
Contributor

@cseickel can you please login to https://codecov.io and allow it to access this repository?

Done. I added the upload token to the repo as CODECOV_TOKEN if you need it.

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@5e9a08b). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #419   +/-   ##
=======================================
  Coverage        ?   52.96%           
=======================================
  Files           ?       48           
  Lines           ?     5405           
  Branches        ?        0           
=======================================
  Hits            ?     2863           
  Misses          ?     2542           
  Partials        ?        0           

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 5e9a08b...c565b36. Read the comment docs.

@MunifTanjim MunifTanjim marked this pull request as ready for review July 12, 2022 11:39
@MunifTanjim
Copy link
Contributor Author

Done. I added the upload token to the repo as CODECOV_TOKEN if you need it.

Awesome! I don't think CODECOV_TOKEN is necessary of public repos.

Anyway, it's working. After this is merged, it should post coverage report on every PR.

@cseickel
Copy link
Contributor

Nice! Thanks!

@cseickel cseickel merged commit 6585cbe into nvim-neo-tree:main Jul 12, 2022
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