Visitar URL original
Link to GitHub repository from documentation landing page by namurphy · Pull Request #1839 · PlasmaPy/PlasmaPy · GitHub
Skip to content

Link to GitHub repository from documentation landing page - #1839

Merged
namurphy merged 4 commits into
PlasmaPy:mainfrom
namurphy:link-to-repo-from-sidebar
Dec 20, 2022
Merged

namurphy merged 4 commits into
PlasmaPy:mainfrom
namurphy:link-to-repo-from-sidebar

Conversation

@namurphy

Copy link
Copy Markdown
Member

This PR adds a link to the GitHub repository in the documentation sidebar for the landing page. Yet another PR inspired by a podcast. 📻

@namurphy namurphy added docs PlasmaPy Docs at http://docs.plasmapy.org no changelog entry needed For PRs that should not have a changelog entry labels Dec 19, 2022
@namurphy namurphy added this to the 2023.1.0 milestone Dec 19, 2022
@github-actions

Copy link
Copy Markdown

Thank you for contributing to PlasmaPy! The project's future depends deeply on contributors like you, so we deeply appreciate it! 🌱 The following checklist will be used by the code reviewer to help guide the code review process.

  • Overall
    • Does the PR do what it intends to do?
    • Except for very minor changes, is a changelog entry included and consistent with the changelog guide?
    • Are the continuous integration checks passing? (Most linter problems can be automagically fixed by commenting on this PR with pre-commit.ci autofix.)
  • Code
    • Is new/updated code understandable and consistent with the coding guide?
    • Are there ways to greatly simplify the implementation?
    • Are there any large functions that should be split up into shorter functions?
  • Tests
    • Are tests added/updated as required, and consistent with the testing guide?
    • Are the tests understandable?
    • Do the tests cover all important cases?
  • Docs
    • Are docs added/updated as required, and consistent with the doc guide?
    • Are the docs understandable?
    • Do the docs show up correctly in the preview, including Jupyter notebooks?

@codecov

codecov Bot commented Dec 19, 2022 •

Copy link
Copy Markdown

Codecov Report

Base: 98.33% // Head: 98.33% // No change to project coverage 👍

Coverage data is based on head (5e26ca0) compared to base (c61da87).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1839   +/-   ##
=======================================
  Coverage   98.33%   98.33%           
=======================================
  Files          95       95           
  Lines        8343     8343           
=======================================
  Hits         8204     8204           
  Misses        139      139           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@namurphy
namurphy marked this pull request as ready for review December 20, 2022 16:29
@namurphy
namurphy requested a review from a team as a code owner December 20, 2022 16:29
@namurphy
namurphy requested review from StanczakDominik and removed request for a team December 20, 2022 16:29
@namurphy
namurphy merged commit 2816852 into PlasmaPy:main Dec 20, 2022
@namurphy
namurphy deleted the link-to-repo-from-sidebar branch December 21, 2022 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs PlasmaPy Docs at http://docs.plasmapy.org no changelog entry needed For PRs that should not have a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants