Skip to content

Commit 1ca05f6

Browse files
committed
resolve conflicts with our config
2 parents 87afabd + 8deadb3 commit 1ca05f6

File tree

458 files changed

+15531
-2863
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

458 files changed

+15531
-2863
lines changed

.editorconfig

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,5 @@ trim_trailing_whitespace = true
88
indent_size = 4
99
indent_style = space
1010

11-
[*.{md,yml,yaml}]
11+
[*.{md,yml,yaml,cff}]
1212
indent_size = 2

.github/CODEOWNERS

+6
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
**/ceci_** @MartinGrignard
2+
**/giga** @MartinGrignard
3+
**/shu_bmrc** @lquayle88
4+
**/utd_ganymede** @edmundmiller @alyssa-ab
5+
**/unsw_katana** @jscgh
6+
**/seadragon** @jiawku

.github/PULL_REQUEST_TEMPLATE.md

+2-1
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,10 @@ Steps for adding a new config profile:
1616
- [ ] Add your custom profile to the `nfcore_custom.config` file in the top-level directory
1717
- [ ] Add your custom profile to the `README.md` file in the top-level directory
1818
- [ ] Add your profile name to the `profile:` scope in `.github/workflows/main.yml`
19+
- [ ] OPTIONAL: Add your custom profile path and GitHub user name to `.github/CODEOWNERS` (`**/<custom-profile>** @<github-username>`)
1920

2021
<!--
21-
If you require/still waiting for a review, please feel free to request from @nf-core/configs-team
22+
If you require/still waiting for a review, please feel free to request a review from @nf-core/maintainers
2223
2324
Please see uploading to`nf-core/configs` for more details:
2425
https://github.com/nf-core/configs#uploading-to-nf-coreconfigs

.github/generate_codeowners.sh

+22
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
#!/usr/bin/env bash
2+
3+
CONFIGS=$(fd . -e .config 'conf/')
4+
5+
output_file=".github/CODEOWNERS"
6+
rm $output_file
7+
8+
for file in $CONFIGS; do
9+
# Get the line of the file that starts with config_profile_contact
10+
line=$(rg "config_profile_contact" "$file")
11+
# and then get the username after the @ and remove anything after it
12+
username=$(echo "$line" | sed 's/^.*@/@/g')
13+
# Remove the )'
14+
username=$(echo "$username" | sed 's/).*$//g')
15+
16+
# Get the insitute name
17+
# conf/<institute>.config
18+
institute=$(echo "$file" | sed 's/^.*\///g' | sed 's/\.config$//g')
19+
20+
# # Remove quotes from authors
21+
echo "**$institute**" $username >> $output_file
22+
done

.github/workflows/fix-linting.yml

+89
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,89 @@
1+
name: Fix linting from a comment
2+
on:
3+
issue_comment:
4+
types: [created]
5+
6+
jobs:
7+
fix-linting:
8+
# Only run if comment is on a PR with the main repo, and if it contains the magic keywords
9+
if: >
10+
contains(github.event.comment.html_url, '/pull/') &&
11+
contains(github.event.comment.body, '@nf-core-bot fix linting') &&
12+
github.repository == 'nf-core/configs'
13+
runs-on: ubuntu-latest
14+
steps:
15+
# Use the @nf-core-bot token to check out so we can push later
16+
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
17+
with:
18+
token: ${{ secrets.nf_core_bot_auth_token }}
19+
20+
# indication that the linting is being fixed
21+
- name: React on comment
22+
uses: peter-evans/create-or-update-comment@71345be0265236311c031f5c7866368bd1eff043 # v4
23+
with:
24+
comment-id: ${{ github.event.comment.id }}
25+
reactions: eyes
26+
27+
# Action runs on the issue comment, so we don't get the PR by default
28+
# Use the gh cli to check out the PR
29+
- name: Checkout Pull Request
30+
run: gh pr checkout ${{ github.event.issue.number }}
31+
env:
32+
GITHUB_TOKEN: ${{ secrets.nf_core_bot_auth_token }}
33+
34+
# Install and run pre-commit
35+
- uses: actions/setup-python@0a5c61591373683505ea898e09a3ea4f39ef2b9c # v5
36+
with:
37+
python-version: 3.11
38+
39+
- name: Install pre-commit
40+
run: pip install pre-commit
41+
42+
- name: Run pre-commit
43+
id: pre-commit
44+
run: pre-commit run --all-files
45+
continue-on-error: true
46+
47+
# indication that the linting has finished
48+
- name: react if linting finished succesfully
49+
if: steps.pre-commit.outcome == 'success'
50+
uses: peter-evans/create-or-update-comment@71345be0265236311c031f5c7866368bd1eff043 # v4
51+
with:
52+
comment-id: ${{ github.event.comment.id }}
53+
reactions: "+1"
54+
55+
- name: Commit & push changes
56+
id: commit-and-push
57+
if: steps.pre-commit.outcome == 'failure'
58+
run: |
59+
git config user.email "core@nf-co.re"
60+
git config user.name "nf-core-bot"
61+
git config push.default upstream
62+
git add .
63+
git status
64+
git commit -m "[automated] Fix code linting"
65+
git push
66+
67+
- name: react if linting errors were fixed
68+
id: react-if-fixed
69+
if: steps.commit-and-push.outcome == 'success'
70+
uses: peter-evans/create-or-update-comment@71345be0265236311c031f5c7866368bd1eff043 # v4
71+
with:
72+
comment-id: ${{ github.event.comment.id }}
73+
reactions: hooray
74+
75+
- name: react if linting errors were not fixed
76+
if: steps.commit-and-push.outcome == 'failure'
77+
uses: peter-evans/create-or-update-comment@71345be0265236311c031f5c7866368bd1eff043 # v4
78+
with:
79+
comment-id: ${{ github.event.comment.id }}
80+
reactions: confused
81+
82+
- name: react if linting errors were not fixed
83+
if: steps.commit-and-push.outcome == 'failure'
84+
uses: peter-evans/create-or-update-comment@71345be0265236311c031f5c7866368bd1eff043 # v4
85+
with:
86+
issue-number: ${{ github.event.issue.number }}
87+
body: |
88+
@${{ github.actor }} I tried to fix the linting errors, but it didn't work. Please fix them manually.
89+
See [CI log](https://github.com/nf-core/configs/actions/runs/${{ github.run_id }}) for more details.

.github/workflows/linting.yml

+13-14
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,21 @@
1-
name: Code Linting
1+
name: Lint tools code formatting
22
on:
3-
pull_request:
43
push:
54
branches:
65
- master
6+
pull_request:
7+
8+
# Cancel if a newer run is started
9+
concurrency:
10+
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
11+
cancel-in-progress: true
712

813
jobs:
9-
prettier:
14+
pre-commit:
1015
runs-on: ubuntu-latest
16+
env:
17+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
1118
steps:
12-
- name: Check out repository
13-
uses: actions/checkout@v2
14-
15-
- name: Install NodeJS
16-
uses: actions/setup-node@v2
17-
18-
- name: Install Prettier
19-
run: npm install -g prettier
20-
21-
- name: Run Prettier --check
22-
run: prettier --check ${GITHUB_WORKSPACE}
19+
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
20+
- uses: actions/setup-python@0a5c61591373683505ea898e09a3ea4f39ef2b9c # v5
21+
- uses: pre-commit/action@v3.0.1

.github/workflows/linting_comment.yml

+28
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
name: nf-core linting comment
2+
# This workflow is triggered after the linting action is complete
3+
# It posts an automated comment to the PR, even if the PR is coming from a fork {%- raw %}
4+
5+
on:
6+
workflow_run:
7+
workflows: ["nf-core linting"]
8+
9+
jobs:
10+
test:
11+
runs-on: ubuntu-latest
12+
steps:
13+
- name: Download lint results
14+
uses: dawidd6/action-download-artifact@f6b0bace624032e30a85a8fd9c1a7f8f611f5737 # v3
15+
with:
16+
workflow: linting.yml
17+
workflow_conclusion: completed
18+
19+
- name: Get PR number
20+
id: pr_number
21+
run: echo "pr_number=$(cat linting-logs/PR_number.txt)" >> $GITHUB_OUTPUT
22+
23+
- name: Post PR comment
24+
uses: marocchino/sticky-pull-request-comment@331f8f5b4215f0445d3c07b4967662a32a2d3e31 # v2
25+
with:
26+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
27+
number: ${{ steps.pr_number.outputs.pr_number }}
28+
path: linting-logs/lint_results.md

.github/workflows/main.yml

+105-13
Original file line numberDiff line numberDiff line change
@@ -2,20 +2,25 @@ name: Configs tests
22

33
on: [pull_request, push]
44

5+
# Cancel if a newer run is started
6+
concurrency:
7+
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
8+
cancel-in-progress: true
9+
510
jobs:
611
test_all_profiles:
712
runs-on: ubuntu-latest
813
name: Check if all profiles are tested
914
steps:
10-
- uses: actions/checkout@v1
15+
- uses: actions/checkout@v4
1116
- name: Check whether profiles are all tested
1217
run: python ${GITHUB_WORKSPACE}/bin/cchecker.py ${GITHUB_WORKSPACE}/nfcore_custom.config ${GITHUB_WORKSPACE}/.github/workflows/main.yml
1318

1419
check_nextflow_config:
1520
runs-on: ubuntu-latest
1621
name: Check if nextflow config runs in repository root
1722
steps:
18-
- uses: actions/checkout@v1
23+
- uses: actions/checkout@v4
1924
- name: Install Nextflow
2025
run: |
2126
wget -qO- get.nextflow.io | bash
@@ -30,67 +35,154 @@ jobs:
3035
matrix:
3136
profile:
3237
- "abims"
38+
- "adcra"
3339
- "alice"
40+
- "apollo"
41+
- "arcc"
3442
- "aws_tower"
3543
- "awsbatch"
3644
- "azurebatch"
3745
- "bi"
3846
- "bigpurple"
47+
- "bih"
3948
- "binac"
4049
- "biohpc_gen"
50+
- "biowulf"
51+
- "bluebear"
4152
- "cambridge"
42-
- "leicester"
4353
- "cbe"
4454
- "ccga_dx"
4555
- "ccga_med"
56+
- "ceci_dragon2"
57+
- "ceci_nic5"
58+
- "cedars"
59+
- "ceres"
4660
- "cfc"
4761
- "cfc_dev"
4862
- "cheaha"
4963
- "computerome"
64+
- "create"
65+
- "crg"
5066
- "crick"
67+
- "crukmi"
68+
- "csiro_petrichor"
69+
- "daisybio"
5170
- "denbi_qbic"
71+
- "dkfz"
5272
- "ebc"
73+
- "ebi_codon"
74+
- "ebi_codon_slurm"
5375
- "eddie"
76+
- "embl_hd"
77+
- "engaging"
78+
- "einstein"
79+
- "ethz_euler"
5480
- "eva"
5581
- "fgcz"
82+
- "fub_curta"
5683
- "genotoul"
5784
- "genouest"
85+
- "giga"
5886
- "gis"
5987
- "google"
88+
- "googlebatch"
89+
- "googlels"
6090
- "hasta"
61-
- "hebbe"
91+
- "hki"
92+
- "hypatia"
6293
- "icr_davros"
6394
- "ifb_core"
95+
- "imb"
96+
- "ilifu"
6497
- "imperial"
98+
- "incliva"
99+
- "ipop_up"
100+
- "janelia"
65101
- "jax"
102+
- "jex"
103+
- "unsw_katana"
104+
- "ku_sund_danhead"
105+
- "kaust"
106+
- "ki_luria"
107+
- "leicester"
66108
- "lugh"
67-
- "marvin"
109+
- "m3c"
68110
- "maestro"
111+
- "mana"
112+
- "marjorie"
113+
- "marvin"
114+
- "mccleary"
115+
- "medair"
116+
- "mjolnir_globe"
69117
- "mpcdf"
118+
- "mpcdf_viper"
70119
- "munin"
120+
- "nci_gadi"
71121
- "nu_genomics"
72-
- "nihbiowulf"
122+
- "nygc"
123+
- "nyu_hpc"
73124
- "oist"
74125
- "pasteur"
126+
- "pawsey_nimbus"
127+
- "pawsey_setonix"
128+
- "pdc_kth"
129+
- "pe2"
75130
- "phoenix"
76-
- "prince"
131+
- "psmn"
132+
- "qmul_apocrita"
77133
- "rosalind"
134+
- "rosalind_uge"
135+
- "roslin"
136+
- "sage"
78137
- "sahmri"
79138
- "sanger"
139+
- "scw"
140+
- "seadragon"
141+
- "seawulf"
80142
- "seg_globe"
143+
- "self_hosted_runner"
144+
- "shu_bmrc"
145+
- "software_license"
146+
- "stjude"
147+
- "tes"
148+
- "tigem"
149+
- "tubingen_apg"
150+
- "tufts"
151+
- "tuos_stanage"
152+
- "ucl_cscluster"
153+
- "ucl_myriad"
81154
- "uct_hpc"
155+
- "ucd_sonic"
156+
- "uge"
82157
- "unibe_ibu"
158+
- "unity"
159+
- "unc_lccc"
160+
- "unc_longleaf"
161+
- "uod_hpc"
83162
- "uppmax"
84163
- "utd_ganymede"
85-
- "utd_sysbio"
164+
- "uw_hyak_pedslabs"
86165
- "uzh"
166+
- "uzl_omics"
87167
- "vai"
168+
- "vsc_calcua"
169+
- "vsc_kul_uhasselt"
170+
- "vsc_ugent"
171+
- "wehi"
172+
- "wustl_htcf"
173+
- "xanadu"
174+
- "york_viking"
175+
88176
steps:
89-
- uses: actions/checkout@v1
90-
- name: Install Nextflow
91-
run: |
92-
wget -qO- get.nextflow.io | bash
93-
sudo mv nextflow /usr/local/bin/
177+
- uses: actions/checkout@v4
178+
- uses: actions/setup-java@8df1039502a15bceb9433410b1a100fbe190c53b # v4
179+
with:
180+
distribution: "temurin"
181+
java-version: "17"
182+
- name: Set up Nextflow
183+
uses: nf-core/setup-nextflow@v2
184+
with:
185+
version: "latest-everything"
94186
- name: Check ${{ matrix.profile }} profile
95187
env:
96188
SCRATCH: "~"

0 commit comments

Comments
 (0)