Thank you for your interest in contributing to Cookie Consent Bundle! This document provides guidelines for contributing to the project.
This project adheres to a code of conduct. By participating, you are expected to uphold this code. Please report unacceptable behavior to hectorfranco@nowo.tech.
- Check existing issues at GitHub Issues
- Create a new issue with steps to reproduce, expected vs. actual behavior, and PHP/Symfony versions
- Search for duplicate suggestions in the issue tracker
- Describe the use case, benefits, and possible implementation
git clone https://github.com/nowo-tech/CookieConsentBundle.git
cd cookie-consent-bundle
make up
make install- PSR-12 via PHP-CS-Fixer
- PHP 8.1+ with
declare(strict_types=1); - PHPDoc in English for public classes and methods
make cs-check
make cs-fix
make phpstan
make rector-drymake test
make test-coverage
make test-ts
make assetsAdd or update tests under tests/Unit/ (and tests/Integration/ when applicable) for behavior changes.
make -C demo up-symfony8
make -C demo up-symfony8-tailwind
make -C demo release-checkAfter editing bundle PHP code in a mounted demo, run make update-bundle inside the demo directory (or rely on release-check, which syncs the bundle first).
Before opening a PR that targets a release:
make release-checkSee Release for maintainer steps.
- Fork and create a feature branch
- Update
docs/CHANGELOG.mdunder[Unreleased]for user-facing changes - Update
docs/UPGRADING.mdwhen consumers must change configuration or code - Ensure
make release-checkpasses locally (or document why not)
- Integrator docs:
docs/INSTALLATION.md,docs/CONFIGURATION.md,docs/USAGE.md - Product spec and
REQ-*traceability:docs/SPEC-DRIVEN-DEVELOPMENT.md
By contributing, you agree that your contributions will be licensed under the MIT License.
Do not add Co-authored-by: Cursor or cursoragent@cursor.com trailers to commit messages.
make setup-hooks
make check-no-cursor-coauthormake setup-hooks installs .githooks/commit-msg (or sets core.hooksPath to .githooks). Run it once per clone before your first commit.
If CI fails because trailers are already on the remote, see GITHUB_CI.md (REQ-GIT-001) and run make strip-cursor-coauthor-from-history before git push --force-with-lease.