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.

Sign up for GitHub

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

Add get_mnv #51178

Merged
merged 13 commits into from
Oct 7, 2024
Merged

Add get_mnv #51178

Changes from 3 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
dce5aa4
add get_mnv
Paururo Oct 5, 2024
64c730c
add get_mnv
Paururo Oct 5, 2024
eeeaa3a
Fix typo, add categories
Paururo Oct 5, 2024
1dcc3da
Add run_exports to get_mnv recipe
Paururo Oct 5, 2024
48bb441
fix run_exports to get_mnv recipe
Paururo Oct 5, 2024
60b8600
fix dependencies
Paururo Oct 5, 2024
628d818
fix dependencies
Paururo Oct 5, 2024
2a25e4a
fix open error
Paururo Oct 5, 2024
7ad6edb
try connect
Paururo Oct 5, 2024
308fc05
Update license
Paururo Oct 6, 2024
3770d9c
libraries to host and add license_fam
Paururo Oct 6, 2024
17df3f1
libraries to host and add license_fam
Paururo Oct 6, 2024
c2af868
add rabit sugg pins in run, license and ver
Paururo Oct 6, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 42 additions & 0 deletions recipes/get_mnv/meta.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
{% set name = "get_mnv" %}
{% set version = "1.0.0" %}
{% set sha256 = "a94eda49def81ef76217b8d39095677335945105d581a0cc9b777db4f6f6b014" %}

package:
name: "{{ name|lower }}"
version: "{{ version }}"

source:
url: "https://github.com/PathoGenOmics-Lab/{{ name }}/archive/refs/tags/{{ version }}.tar.gz"
sha256: "{{ sha256 }}"

build:
number: 0
script: "cargo install --no-track --locked --verbose --root \"${PREFIX}\" --path ."

requirements:
build:
- {{ compiler('rust') }}

test:
commands:
- get_mnv --help # Asegúrate de que `get_mnv` sea el nombre del binario
Copy link
Contributor

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Enhance test coverage and maintain consistency.

While the current test ensures that the get_mnv binary is installed and executable, consider adding more comprehensive tests to verify the tool's functionality.

  1. Translate the Spanish comment to English for consistency:
-    - get_mnv --help  # Asegúrate de que `get_mnv` sea el nombre del binario
+    - get_mnv --help  # Ensure that `get_mnv` is the correct binary name
  1. Consider adding more test commands to verify key functionalities of the tool. For example:
test:
  commands:
    - get_mnv --help
    - get_mnv --version
    - get_mnv --test-data  # If there's a test data option

Would you like assistance in generating more comprehensive test commands based on the tool's functionality?



about:
home: https://github.com/PathoGenOmics-Lab/get_mnv
license: GPL-3.0
license_file: LICENSE
summary: 'Tool to identify Multi-Nucleotide Variants (MNVs) in genomic sequences.'
description: |
get_MNV is a tool designed to identify Multi-Nucleotide Variants (MNVs) within the same codon in genomic sequences, providing more accurate annotation for genomic data.
doc_url: https://github.com/PathoGenOmics-Lab/get_mnv
dev_url: https://github.com/PathoGenOmics-Lab/get_mnv

extra:
recipe-maintainers:
- PathoGenOmics-Lab
categories:
- Genomics
- Variant Analysis

Loading