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

update link on documentation #1137

Merged

Conversation

lacoak21
Copy link
Contributor

@lacoak21 lacoak21 commented Nov 8, 2024

Change Summary

closes #1134

Overview

Update the "other" documentation in the getting started page.

Updated Files

  • docs/source/development-guide/getting-started.rst
    • Removed the link to the galaxy help page
    • Updated wording to be more clear.

@lacoak21 lacoak21 added the Repo: Documentation Improvements or additions to documentation label Nov 8, 2024
@lacoak21 lacoak21 self-assigned this Nov 8, 2024
Copy link
Contributor

@nkerman nkerman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This changes everything. Absolutely flawless code.

@@ -28,8 +28,7 @@ Our tests are run using pytest:
poetry shell
pytest

Other Information
Related Information
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great change!

For more information relating to infrastructure, see `sds-data-manager <https://sds-data-manager.readthedocs.io/en/latest/>`_.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Copy link
Contributor

@torimarbois torimarbois left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

flawless

Copy link
Contributor

@tech3371 tech3371 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First PR!

@lacoak21 lacoak21 merged commit f3c95f3 into IMAP-Science-Operations-Center:dev Nov 8, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Repo: Documentation Improvements or additions to documentation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Remove unused link from document
4 participants