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

Fix CI: Add missing requirements, build Caddy with cloudflare module #6

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
30 changes: 29 additions & 1 deletion .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,32 @@ on: [workflow_dispatch, pull_request, push]
jobs:
test:
runs-on: ubuntu-latest
steps: [uses: fastai/workflows/nbdev-ci@master]
steps:
- name: Install Go
uses: actions/setup-go@v4
with:
go-version: '>=1.21.0'

- name: Install xcaddy
run: go install github.com/caddyserver/xcaddy/cmd/xcaddy@latest

- name: Build caddy with cloudflare plugin
run: |
cd ~/go/bin
./xcaddy build --with github.com/caddy-dns/cloudflare

- name: Start caddy for testing
run: |
~/go/bin/caddy run &
sleep 5 # Give caddy time to start

- name: Run nbdev tests
env:
CLOUDFLARE_API_TOKEN: DUMMY_TOKEN
uses: fastai/workflows/nbdev-ci@master
with:
flags: "--skip_file_re \"index|test_\""

- name: Stop caddy
run: pkill caddy

2 changes: 1 addition & 1 deletion settings.ini
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ version = 0.0.4
min_python = 3.7
license = apache2
black_formatting = False
requirements = fastcore httpx
requirements = fastcore httpx cloudflare rich
doc_path = _docs
lib_path = fastcaddy
nbs_path = nbs
Expand Down
Loading