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 GCC7 hint #14395

Merged
merged 4 commits into from
Apr 14, 2019
Merged

remove GCC7 hint #14395

merged 4 commits into from
Apr 14, 2019

Conversation

bgruening
Copy link
Member

@bioconda/core I think this note can be removed now?

ℹ️
Bioconda has finished the GCC7 migration. If you are dealing with C/C++ or Python package it can be that you need to rebuild other dependent packages. Bioconda utils - update-pinning will assist you with that. If you have any questions please use issue 13578.


  • 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).

@dpryan79
Copy link
Contributor

dpryan79 commented Apr 9, 2019

Yup, we should put in the change about pinging core to get added to the project. Shall I put that in your PR or make a separate one?

@biocondabot
Copy link
Contributor

biocondabot bot commented Apr 9, 2019

CirleCI builds completed without yielding any package artifacts.

@bgruening
Copy link
Member Author

Feel free to take this over!

@dpryan79
Copy link
Contributor

dpryan79 commented Apr 9, 2019

There's a first stab at things, it'd be nice if someone looked it over. I'll mention this in the "Contributing" section of the documentation.

@@ -1,5 +1,5 @@
:information_source:
Bioconda has finished the [GCC7 migration](https://github.com/bioconda/bioconda-recipes/issues/13578). If you are dealing with C/C++ or Python package it can be that you need to rebuild other dependent packages. [Bioconda utils - update-pinning](https://bioconda.github.io/updating.html#updating-recipes-for-a-pinning-change) will assist you with that. If you have any questions please use [issue 13578](https://github.com/bioconda/bioconda-recipes/issues/13578).
If this is your first pull request to Bioconda please ping @bioconda/core so it can be reviewed and to request being added as a member of the bioconda team.
Copy link
Member Author

Choose a reason for hiding this comment

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

I'm wondering if this note will now ping us every time :)

Maybe we should explain what it means to be added to the bioconda-team? We can also link to docs I guess.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, I'm not sure if we'll get auto-pinged or not. If so we have to change that. Yes, maybe we should update the docs first.

@bgruening bgruening merged commit 75f4fc4 into master Apr 14, 2019
@bgruening
Copy link
Member Author

Danke!

@bgruening bgruening deleted the gcc7-hint branch April 14, 2019 08:21
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.

2 participants