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

flash bump #9088

Closed
wants to merge 2 commits into from
Closed

flash bump #9088

wants to merge 2 commits into from

Conversation

mictadlo
Copy link
Contributor

  • I have read the guidelines for bioconda recipes.
  • This PR adds a new recipe.
  • AFAIK, this recipe is directly relevant to the biological sciences (otherwise, please submit to the more general purpose conda-forge channel).
  • This PR updates an existing recipe.
  • This PR does something else (explain below).

@mictadlo mictadlo requested review from bgruening, ArneKr and marcelm and removed request for bgruening and ArneKr May 31, 2018 00:10

extra:
Copy link
Member

Choose a reason for hiding this comment

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

Why have you removed this? its still the same tool isn't it?
Is there a publication which we can reference?

version: {{ version }}

build:
number: 1
Copy link
Member

Choose a reason for hiding this comment

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

start with 0

@marcelm
Copy link
Contributor

marcelm commented May 31, 2018

I’m uncertain as to the status of how FLASH2 relates to FLASH. Looking at the GitHub repository, it seems that the project is an independent one (not from the original authors).

Perhaps @dstreett could explain?

According to dstreett/FLASH2#2, the behavior is also slightly different, so perhaps it woud be an idea to put this into a separate recipe named flash2.

@bgruening
Copy link
Member

It it is really a separate "tool" than a new recipe would be preferred, I agree.

@mictadlo
Copy link
Contributor Author

I created a new PR #9103

@mictadlo mictadlo closed this May 31, 2018
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