Skip to content
This repository has been archived by the owner on Oct 14, 2023. It is now read-only.

Point README COPYING link to main; not master #1121

Merged
merged 1 commit into from
Mar 18, 2021

Conversation

xlxs4
Copy link
Contributor

@xlxs4 xlxs4 commented Mar 16, 2021

As per #1119.

@codecov
Copy link

codecov bot commented Mar 16, 2021

Codecov Report

Merging #1121 (3570d5f) into main (c5106d6) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1121   +/-   ##
=======================================
  Coverage   90.02%   90.02%           
=======================================
  Files          73       73           
  Lines        3861     3861           
  Branches      330      330           
=======================================
  Hits         3476     3476           
  Misses        297      297           
  Partials       88       88           

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 c5106d6...3570d5f. Read the comment docs.

@xlxs4
Copy link
Contributor Author

xlxs4 commented Mar 16, 2021

Couldn't understand exactly why requests failed, shouldn't have to do with the PR

@jorgepiloto jorgepiloto merged commit 56cc3be into poliastro:main Mar 18, 2021
@jorgepiloto
Copy link
Member

Hi @xlxs4,

thanks for addressing this. You are right, the error is not related to the actual changes but to some "timeout" exception when requesting data from NASA server. Thus, let us merge this 🚀

@xlxs4 xlxs4 deleted the readme-main-rename-fix branch March 18, 2021 14:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants