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 usage and installation docs, point to new CLI docs on README #453

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

llewellyn-sl
Copy link
Contributor

Strip out documentation content here and link to new CLI docs on docs.seqera.io

Copy link
Contributor

@pditommaso pditommaso left a comment

Choose a reason for hiding this comment

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

Keep the usage here in the GH repo please

@llewellyn-sl
Copy link
Contributor Author

@pditommaso usage.md is outdated compared to the Commands page in the docs. Keeping both pages updated is an additional maintenance burden. The Platform docs page contains everything the usage page here had, with more examples and all of the missing commands. Do we need to keep this page simply for posterity?

@pditommaso
Copy link
Contributor

pditommaso commented Sep 11, 2024

Full usage I agree, however as OSS user/developer I expect to find the most important into here. The "Get started", "Installation" and basic commands should be reported in the *README.md* file.

@llewellyn-sl llewellyn-sl dismissed pditommaso’s stale review September 20, 2024 08:14

Requested changes addressed.

Copy link
Member

@robnewman robnewman left a comment

Choose a reason for hiding this comment

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

Looks good! Minor changes

@@ -4,175 +4,96 @@

Seqera Platform is a full-stack application for the management of data pipelines and compute resources. It enables collaborative data analysis at scale, on-premises or in any cloud.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Seqera Platform is a full-stack application for the management of data pipelines and compute resources. It enables collaborative data analysis at scale, on-premises or in any cloud.
Seqera Platform is a full-stack application for the management of data pipelines, compute resources and tertiary analysis. It enables collaborative data analysis at scale, on-premises or in any cloud.


![tw](assets/img/tw-screenshot.png)

The key features are:
See the [Platform CLI documentation](https://docs.seqera.io/platform/24.1/cli/overview).
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
See the [Platform CLI documentation](https://docs.seqera.io/platform/24.1/cli/overview).
See the [Platform CLI documentation](https://docs.seqera.io/platform/24.1/cli/overview) for complete functionality.


- **A Nextflow-like experience**: tw CLI provides a developer-friendly environment. Pipelines can be launched with the CLI similarly to Nextflow but with the Seqera benefits of monitoring, logging, resource provisioning, dataset management, and collaborative sharing.
### Installation
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
### Installation
## Quick-start Guide
### Installation

Copy link
Member

Choose a reason for hiding this comment

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

I think this change will also resolve the linting errors.

@pditommaso
Copy link
Contributor

We should make in case the README self-contained and allow the user to get start without the need to go in the docs. Therefore (core) installation info should stay here (same for all OSS projects)

@robnewman
Copy link
Member

robnewman commented Sep 20, 2024

We should make in case the README self-contained and allow the user to get start without the need to go in the docs. Therefore (core) installation info should stay here (same for all OSS projects)

I believe the core installation instructions remain in this updated PR, but agree that basic usage would be helpful.

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.

3 participants