Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Remove Travis configuration #87

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

Merged
merged 1 commit into from
Aug 11, 2019
Merged

Conversation

astrofrog
Copy link
Member

I've now set up Azure Pipelines to do a weekly cron job (ideally I wanted to do monthly but the UI doesn't allow that yet). There is also a cron job defined in .azure-pipelines.yml but that isn't yet taken into account due to an Azure bug. In any case, now that there is a cron job set up on Azure, I think we can remove the Travis configuration for simplicity.

@codecov
Copy link

codecov bot commented Aug 10, 2019

Codecov Report

Merging #87 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master     #87   +/-   ##
======================================
  Coverage    87.1%   87.1%           
======================================
  Files           5       5           
  Lines         721     721           
======================================
  Hits          628     628           
  Misses         93      93

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 62e6737...708f4bd. Read the comment docs.

@bsipocz
Copy link
Member

bsipocz commented Aug 10, 2019

I think weekly is OK, having a monthly might not be frequent enough to catch issues in time (I suppose I would most prefer biweekly).

@astrofrog astrofrog merged commit ccf9354 into astropy:master Aug 11, 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