Skip to content

Conversation

sjvrijn
Copy link
Contributor

@sjvrijn sjvrijn commented Aug 20, 2024

Description

Adds a gitlab-ci.yml file to provide similar functionality as the current GitHub workflows:

  • Automated tests on all supported Python versions, but only on Linux
  • CFF convert validation job (should only run if CITATION.cff file is changed)
  • Lychee link checker. I didn't get around to adding the cron-job running, or the issue creation based on link errors from it.

See this GitLab repo's CI output for output examples. Tests pass, CFF convert and links checker fail as expected for a default generated package.

Note changelog entry is still lumped in with 'unreleased' entries belonging to v0.5.0. See #455

Related issues:

Instructions to review the pull request

Create a python-template-test repo on GitLab (will be overwritten if existing)

# Create a temporary directory by running the following command. Keep the XXXXXX in the directory name. 
cd $(mktemp -d --tmpdir py-tmpl-XXXXXX)
# Use --vcs-ref <pr-branch> to point to the branch you want to test
copier copy --vcs-ref 452-gitlab-ci-workflow https://github.com/NLeSC/python-template .
# Fill with python-template-test info
# Create a local git repo to push to GitLab to trigger CI actions
git init
git add --all
git commit -m "First commit"
git remote add origin git@gitlab.com:<you>/python-template-test.git
git push -u origin main -f

@sjvrijn sjvrijn requested review from fdiblen and lyashevska August 21, 2024 00:25
@sjvrijn sjvrijn force-pushed the 452-gitlab-ci-workflow branch from 1e72a26 to 0bd8534 Compare August 21, 2025 15:06
@sjvrijn sjvrijn force-pushed the 452-gitlab-ci-workflow branch from 0bd8534 to b91710e Compare September 5, 2025 21:54
@sjvrijn sjvrijn marked this pull request as draft September 7, 2025 21:45
@sjvrijn sjvrijn force-pushed the 452-gitlab-ci-workflow branch from b91710e to 053c437 Compare September 8, 2025 12:12
@sjvrijn sjvrijn force-pushed the 452-gitlab-ci-workflow branch from 053c437 to 09a1afa Compare September 9, 2025 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants