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

Simple bash action #179

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 1 commit
Commits
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
2 changes: 0 additions & 2 deletions .actrc
Copy link
Member

Choose a reason for hiding this comment

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

.actrc are the directions to act to allow for local testing of this action. If this file isn't present then two things will happen:

  1. act will attempt to download a 50+ GB docker image to run tests
  2. The workflows will fail on Apple silicon systems

Keep this file and add documentation on how to leverage act in local testing

This file was deleted.

3 changes: 0 additions & 3 deletions .eslintignore

This file was deleted.

60 changes: 0 additions & 60 deletions .eslintrc.json

This file was deleted.

1 change: 0 additions & 1 deletion .gitattributes

This file was deleted.

17 changes: 0 additions & 17 deletions .github/renovate.json5

This file was deleted.

42 changes: 7 additions & 35 deletions .github/workflows/example.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@ on:
push:
branches:
- master
# https://docs.renovatebot.com/key-concepts/automerge/#branch-vs-pr-automerging
- "renovate/**" # branches Renovate creates
pull_request:
workflow_dispatch:
release:
Expand All @@ -27,41 +25,15 @@ jobs:
- "latest-stable"
- "latest-edge"
- "latest-everything"
all_distribution:
- true
- false

steps:
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
- uses: actions/setup-node@39370e3970a6d050c480ffad4ff0ed4d3fdee5af # v4
with:
node-version: 20
cache: "npm"
- run: npm ci
- run: npm run build
- run: npm run package
- uses: ./
- name: Checkout repo
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4

- name: Run action
uses: ./
with:
version: ${{ matrix.nextflow_version }}
all: ${{ matrix.all_distribution }}
- run: nextflow -v

example-maximized-build-space:
runs-on: ubuntu-latest
steps:
- uses: easimon/maximize-build-space@fc881a613ad2a34aca9c9624518214ebc21dfc0c # v10
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
- uses: actions/setup-node@39370e3970a6d050c480ffad4ff0ed4d3fdee5af # v4
with:
node-version: 20
cache: "npm"
- run: npm ci
- run: npm run build
- run: npm run package
- uses: actions/setup-java@8df1039502a15bceb9433410b1a100fbe190c53b # v4
if: ${{ env.ACT}}
with:
java-version: 11
distribution: zulu
- uses: ./
- run: nextflow -v
- name: Check Nextflow version
run: nextflow -v
22 changes: 0 additions & 22 deletions .github/workflows/publish.yml

This file was deleted.

33 changes: 0 additions & 33 deletions .github/workflows/test.yml

This file was deleted.

138 changes: 0 additions & 138 deletions .gitignore

This file was deleted.

1 change: 0 additions & 1 deletion .husky/.gitignore

This file was deleted.

2 changes: 0 additions & 2 deletions .husky/pre-commit
Copy link
Member

Choose a reason for hiding this comment

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

Husky is how we enforced code style in this repo. It's basically like pre-commit, but baked into npm. I think keeping npm around in a non-Javascript project just for commit hooks is ridiculous, but I think it would be better to replace it with pre-commit to be consistent with the rest of nf-core infrastructure.

This file was deleted.

3 changes: 0 additions & 3 deletions .prettierignore

This file was deleted.

10 changes: 0 additions & 10 deletions .prettierrc.json

This file was deleted.

21 changes: 0 additions & 21 deletions .vscode/launch.json

This file was deleted.

Loading