Skip to content
This repository has been archived by the owner on Aug 20, 2024. It is now read-only.

Add deprecation message to the nf-validation docs #163

Closed
wants to merge 4 commits into from

Conversation

nvnieuwk
Copy link
Collaborator

No description provided.

@nvnieuwk nvnieuwk requested review from ewels and mirpedrol March 14, 2024 12:12
Copy link
Collaborator

@mirpedrol mirpedrol left a comment

Choose a reason for hiding this comment

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

LGTM!

@nvnieuwk
Copy link
Collaborator Author

I'll maybe wait a bit until we coordinate the work better so we can keep the limbo time between the two plugins as short as possible

@nvnieuwk nvnieuwk mentioned this pull request Apr 3, 2024
Copy link
Member

@ewels ewels left a comment

Choose a reason for hiding this comment

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

Why do you need to rename the GitHub actions workflow? I'd prefer to leave that in place, in case we want to add any more notices to the docs in the future.

  • Please copy the warning to the docs homepage, in an admonition
  • Please add an announcement bar to the docs, visible on every page

We also need to think about how / if to port all issues and PRs over to the nf-schema repository. Once that's done and this PR is merged, we can archive the repo.

Comment on lines +3 to +7
> [!IMPORTANT] **`nf-validation` has now been deprecated and won't be updated anymore.**
>
> **Please update your pipelines to use [`nf-schema`](https://github.com/nextflow-io/nf-schema) instead.**
>
> This change was necessary to prevent older versions of `nf-core` pipelines from breaking when updating to the latest version of `nf-validation`. Some older and current versions of the pipelines did not pin the version of nf-validation, which caused the pipelines to break when the latest version of nf-validation was released. This change will prevent this from happening in the future. **Please pin the version of `nf-schema` in your pipeline's `nextflow.config` file.**
Copy link
Member

Choose a reason for hiding this comment

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

This doesn't render properly. The admonition type needs to be on it's own line.

Suggested change
> [!IMPORTANT] **`nf-validation` has now been deprecated and won't be updated anymore.**
>
> **Please update your pipelines to use [`nf-schema`](https://github.com/nextflow-io/nf-schema) instead.**
>
> This change was necessary to prevent older versions of `nf-core` pipelines from breaking when updating to the latest version of `nf-validation`. Some older and current versions of the pipelines did not pin the version of nf-validation, which caused the pipelines to break when the latest version of nf-validation was released. This change will prevent this from happening in the future. **Please pin the version of `nf-schema` in your pipeline's `nextflow.config` file.**
> [!IMPORTANT]
> **`nf-validation` has now been renamed to `nf-schema`. The `nf-validation` plugin will not receive any future updates.**
> **Please update your pipelines to use [`nf-schema`](https://github.com/nextflow-io/nf-schema) instead.**
>
> This change was necessary to prevent older versions of `nf-core` pipelines from with unpinned plugin references from breaking when updating to the latest version of `nf-validation`.
> **Please pin the version of `nf-schema` in your pipeline's `nextflow.config` file:**
> ```nextflow
> plugins { id '[email protected]' }
> ```

@ewels
Copy link
Member

ewels commented Apr 22, 2024

Should also point to the migration guide.

@nvnieuwk
Copy link
Collaborator Author

Closing in favor of #170

@nvnieuwk nvnieuwk closed this Apr 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants