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

Remove .readthedocs.yml #1312

Merged
merged 1 commit into from
Jul 21, 2023
Merged

Remove .readthedocs.yml #1312

merged 1 commit into from
Jul 21, 2023

Conversation

AA-Turner
Copy link
Collaborator

cc: @MridulS

This removes a no-longer-needed configuration file.

A

@codecov
Copy link

codecov bot commented Jul 21, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (2209109) 72.55% compared to head (1a69ce6) 72.55%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1312   +/-   ##
=======================================
  Coverage   72.55%   72.55%           
=======================================
  Files          78       78           
  Lines       13009    13009           
=======================================
  Hits         9439     9439           
  Misses       3570     3570           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@MridulS
Copy link
Member

MridulS commented Jul 21, 2023

Should we wait till we have got all the desired changes in?

@AA-Turner
Copy link
Collaborator Author

RTD builds are currently failing -- this PR wouldn't remove the RTD site, but only configuration for already-failing builds.

If desired, we could try and fix RTD?

A

@MridulS
Copy link
Member

MridulS commented Jul 21, 2023

If desired, we could try and fix RTD?

Ah no, let's not invest too much time there. Let's move to docs.econ-ark.org as soon as possible :)

@MridulS MridulS merged commit e552e7e into econ-ark:master Jul 21, 2023
16 checks passed
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