Skip to content

Commit b916b6d

Browse files
authored
Merge pull request nf-core#629 from jfy133/dev
Fix dockerhub issue
2 parents 06b596a + 3997285 commit b916b6d

3 files changed

Lines changed: 14 additions & 31 deletions

File tree

.github/workflows/ci.yml

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -186,34 +186,3 @@ jobs:
186186
- name: MTNUCRATIO Run basic pipeline with bam input profile, but don't convert BAM, skip everything but nmtnucratio
187187
run: |
188188
nextflow run ${GITHUB_WORKSPACE} -profile test_tsv_humanbam,docker --skip_fastqc --skip_adapterremoval --skip_deduplication --skip_qualimap --skip_preseq --skip_damage_calculation --run_mtnucratio
189-
190-
push_dockerhub:
191-
name: Push new Docker image to Docker Hub
192-
runs-on: ubuntu-latest
193-
# Only run if the tests passed
194-
needs: test
195-
# Only run for the nf-core repo, for releases and merged PRs
196-
if: ${{ github.repository == 'nf-core/eager' && (github.event_name == 'release' || github.event_name == 'push') }}
197-
env:
198-
DOCKERHUB_USERNAME: ${{ secrets.DOCKERHUB_USERNAME }}
199-
DOCKERHUB_PASS: ${{ secrets.DOCKERHUB_PASS }}
200-
steps:
201-
- name: Check out pipeline code
202-
uses: actions/checkout@v2
203-
204-
- name: Build new docker image
205-
run: docker build --no-cache . -t nfcore/eager:latest
206-
207-
- name: Push Docker image to DockerHub (dev)
208-
if: ${{ github.event_name == 'push' }}
209-
run: |
210-
echo "$DOCKERHUB_PASS" | docker login -u "$DOCKERHUB_USERNAME" --password-stdin
211-
docker tag nfcore/eager:latest nfcore/eager:dev
212-
docker push nfcore/eager:dev
213-
- name: Push Docker image to DockerHub (release)
214-
if: ${{ github.event_name == 'release' }}
215-
run: |
216-
echo "$DOCKERHUB_PASS" | docker login -u "$DOCKERHUB_USERNAME" --password-stdin
217-
docker push nfcore/eager:latest
218-
docker tag nfcore/eager:latest nfcore/eager:${{ github.ref }}
219-
docker push nfcore/eager:${{ github.ref }}

CHANGELOG.md

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,19 @@
33
The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
44
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
55

6+
## [dev] - Unreleased
7+
8+
### `Added`
9+
10+
### `Fixed`
11+
12+
- Removed leftover old DockerHub push CI commands.
13+
- [#627](https://github.com/nf-core/eager/issues/627) Added de Barros Damgaard citation to README
14+
15+
### `Dependencies`
16+
17+
### `Deprecated`
18+
619
## [2.2.2] - 2020-12-09
720

821
### `Added`

README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -244,3 +244,4 @@ This repository uses test data from the following studies:
244244
* Fellows Yates, J. A. et al. (2017) ‘Central European Woolly Mammoth Population Dynamics: Insights from Late Pleistocene Mitochondrial Genomes’, Scientific reports, 7(1), p. 17714. [doi: 10.1038/s41598-017-17723-1](https://doi.org/10.1038/s41598-017-17723-1).
245245
* Gamba, C. et al. (2014) ‘Genome flux and stasis in a five millennium transect of European prehistory’, Nature communications, 5, p. 5257. [doi: 10.1038/ncomms6257](https://doi.org/10.1038/ncomms6257).
246246
* Star, B. et al. (2017) ‘Ancient DNA reveals the Arctic origin of Viking Age cod from Haithabu, Germany’, Proceedings of the National Academy of Sciences of the United States of America, 114(34), pp. 9152–9157. [doi: 10.1073/pnas.1710186114](https://doi.org/10.1073/pnas.1710186114).
247+
* de Barros Damgaard, P. et al. (2018). '137 ancient human genomes from across the Eurasian steppes.', Nature, 557(7705), 369–374. [doi: 10.1038/s41586-018-0094-2](https://doi.org/10.1038/s41586-018-0094-2)

0 commit comments

Comments
 (0)