Skip to content

Comments

Migrate CI/CD pipeline to GitHub Actions#459

Merged
benadida merged 2 commits intomasterfrom
claude/migrate-github-actions-aXnvq
Jan 17, 2026
Merged

Migrate CI/CD pipeline to GitHub Actions#459
benadida merged 2 commits intomasterfrom
claude/migrate-github-actions-aXnvq

Conversation

@benadida
Copy link
Owner

Replace outdated Travis CI configuration with modern GitHub Actions workflow:

  • Use Python 3.12 (matching runtime.txt)
  • Test against PostgreSQL 14, 15, and 16
  • Add pip caching for faster builds
  • Install required LDAP system dependencies

Replace outdated Travis CI configuration with modern GitHub Actions workflow:
- Use Python 3.12 (matching runtime.txt)
- Test against PostgreSQL 14, 15, and 16
- Add pip caching for faster builds
- Install required LDAP system dependencies
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR modernizes the CI/CD pipeline by migrating from Travis CI to GitHub Actions, updating Python from 3.7/3.8 to 3.12, and PostgreSQL from versions 9.5-11 to 14-16. The new workflow includes pip caching and proper LDAP system dependencies.

Changes:

  • Removed outdated Travis CI configuration (.travis.yml)
  • Added new GitHub Actions workflow with matrix testing across PostgreSQL 14, 15, and 16
  • Configured pip caching and LDAP system dependencies (libldap2-dev, libsasl2-dev)

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.travis.yml Removed legacy Travis CI configuration that tested Python 3.7/3.8 with PostgreSQL 9.5-11
.github/workflows/ci.yml Added modern GitHub Actions workflow using Python 3.12, PostgreSQL 14-16, with pip caching and LDAP dependencies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pip freeze

- name: Run tests
run: python -Wall manage.py test -v 2
Copy link

Copilot AI Jan 17, 2026

Choose a reason for hiding this comment

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

The Python flag syntax is incorrect. The flag should be "-W all" (with a space) rather than "-Wall". The "-Wall" syntax is from GCC/C compilers, not Python. This will cause the test command to fail. The correct command should be: python -W all manage.py test -v 2

Suggested change
run: python -Wall manage.py test -v 2
run: python -W all manage.py test -v 2

Copilot uses AI. Check for mistakes.
The main settings.py ignores DATABASE_URL during tests, causing Django
to attempt Unix socket connection which doesn't work with containerized
PostgreSQL. Add settings_ci.py to provide explicit TCP connection config
for the CI environment.
@benadida benadida merged commit 9d631a4 into master Jan 17, 2026
3 checks passed
@benadida benadida deleted the claude/migrate-github-actions-aXnvq branch January 17, 2026 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants