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

docs: remove duplicate readthedocs yaml #76

Merged
merged 2 commits into from
May 22, 2023
Merged

Conversation

tigarmo
Copy link
Collaborator

@tigarmo tigarmo commented May 22, 2023

This happened during the starbase merge; starbase uses .readthedocs.yaml, but we already had a .readthedocs.yml. Let's prefer upstream's.

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox?

This happened during the starbase merge; starbase uses .readthedocs.yaml,
but we already had a .readthedocs.yml. Let's prefer upstream's.
@codecov
Copy link

codecov bot commented May 22, 2023

Codecov Report

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

❗ Current head 8bcb35c differs from pull request most recent head de0ff99. Consider uploading reports for the commit de0ff99 to get more accurate results

@@           Coverage Diff           @@
##             main      #76   +/-   ##
=======================================
  Coverage        ?   88.09%           
=======================================
  Files           ?       13           
  Lines           ?      630           
  Branches        ?      125           
=======================================
  Hits            ?      555           
  Misses          ?       61           
  Partials        ?       14           

@lengau lengau merged commit fd9fac8 into main May 22, 2023
@lengau lengau deleted the duplicate-readthedocs-yaml branch May 22, 2023 22:49
tigarmo pushed a commit to tigarmo/craft-archives that referenced this pull request May 23, 2023
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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