Skip to content

Latest commit

 

History

History
154 lines (104 loc) · 4.92 KB

CONTRIBUTING.md

File metadata and controls

154 lines (104 loc) · 4.92 KB
Raw

Contributing

Contributions are welcome. This project accepts pull requests on GitHub.

This project adheres to a code of conduct. By participating in this project and its community, you are expected to uphold this code.

Communication Channels

You can find help and discussion in the following places:

Reporting Bugs

Report bugs using the project's issue tracker.

⚠️ ATTENTION!!! DO NOT include passwords or other sensitive information in your bug report.

When submitting a bug report, please include enough information to reproduce the bug. A good bug report includes the following sections:

  • Description

    Provide a short and clear description of the bug.

  • Steps to reproduce

    Provide steps to reproduce the behavior you are experiencing. Please try to keep this as short as possible. If able, create a reproducible script outside of any framework you are using. This will help us to quickly debug the issue.

  • Expected behavior

    Provide a short and clear description of what you expect to happen.

  • Screenshots or output

    If applicable, add screenshots or program output to help explain your problem.

  • Environment details

    Provide details about the system where you're using this package, such as PHP version and operating system.

  • Additional context

    Provide any additional context that may help us debug the problem.

Fixing Bugs

This project welcomes pull requests to fix bugs!

If you see a bug report that you'd like to fix, please feel free to do so. Following the directions and guidelines described in the "Adding New Features" section below, you may create bugfix branches and send pull requests.

Adding New Features

If you have an idea for a new feature, it's a good idea to check out the issues or active pull requests first to see if anyone is already working on the feature. If not, feel free to submit an issue first, asking whether the feature is beneficial to the project. This will save you from doing a lot of development work only to have your feature rejected. We don't enjoy rejecting your hard work, but some features don't fit with the goals of the project.

When you do begin working on your feature, here are some guidelines to consider:

  • Your pull request description should clearly detail the changes you have made. We will use this description to update the CHANGELOG. If there is no description, or it does not adequately describe your feature, we may ask you to update the description.
  • c-base/cbag3 follows a superset of PSR-12 coding standard. Please ensure your code does, too. Hint: run make ci to check.
  • Please write tests for any new features you add.
  • Please ensure that tests pass before submitting your pull request. c-base/cbag3 automatically runs tests for pull requests. However, running the tests locally will help save time. Hint: run make test.
  • Submit one feature per pull request. If you have multiple features you wish to submit, please break them into separate pull requests.
  • Write good commit messages. This project kind of follows the Conventional Commits specification and uses Github actions.

Developing

To develop this project, you will need PHP 8.1 or greater and Composer.

After cloning this repository locally, execute the following commands:

cd /path/to/repository
make install

Commands

To see all the commands available for contributing to this project:

make help

Coding Standards

This project follows a superset of PSR-12 coding standards, enforced by PHP Coding Standards Fixer.

# Lint
make lint-php

# Attempt to auto-fix coding standards issues
make lint-php-fix

Static Analysis

This project uses a combination of PHPStan and maybe others to provide static analysis of PHP code.

You may run static analysis manually across the whole codebase with the following command:

make analyze

Project Structure

This project uses pds/skeleton as its base folder structure and layout.

Running Tests

The following must pass before we will accept a pull request. If this does not pass, it will result in a complete build failure. Before you can run this, be sure to run make install.

To run all the tests and coding standards checks, execute the following from the command line, while in the project root directory:

make test