open-fpsz/CONTRIBUTING.md

[[TOC]]

How to Contribute?

Thank you for considering contributing to our project!

Whether you're an experienced developer or just starting out, we welcome your contributions to help make our game even better.

Reporting Issues

If you encounter any bugs or issues, please report them in the Issue Tracker.

Include as much detail as possible, such as steps to reproduce the issue, expected vs. actual behavior, screenshots or logs.

Merge Requests and Releases

When starting work on a new feature or bug fix, developers should always use the develop branch as their starting point.

The develop branch contains the most recent and advanced codebase, as it serves as a staging area for upcoming features and fixes.

Periodically, changes from develop are merged into the main branch to package releases.

This practice ensures that everyone remains updated on ongoing tasks, fostering transparency and encouraging collaboration.

Learn the various ways to create a merge request.

flowchart LR
    subgraph Alice's Fork
        A[feat/alice-branch] --> C((Merge Request))
    end
    subgraph Bob's Fork
        B[feat/bob-branch] --> D((Merge Request))
    end
    subgraph open-fpsz
        C -->|To Target Branch| E[develop]
        D -->|To Target Branch| E[develop]
        E -->|Merge into\nOnce Stable & Ready| G[main]
    end

Fork-based workflow

If you want to contribute changes without write access to the repository, you can still do so by forking. This is a simple way for anyone to contribute by creating your own copy of the project where you can make changes freely.

1. First, you'll need to fork the repository.
2. Once you have a fork, clone it to your local machine and create a new branch from the develop branch.
3. Then, you can make changes just like you would if you were a member of the project.
4. After you've made your changes, you commit them locally and push these back to your fork on gitlab.
5. Finally, you can create a merge request from your fork's branch to upstream's branch.

We encourage you to allow commits from upstream members so that they can push to the fork as the upstream member in order for you to reach out for help.

The merge request notifies project maintainers that you've made changes and allows them to review your code and potentially merge it into the main project.

When your branch is merged, you may want to update your fork to be synced with upstream.

Repository membership

Access to our gitlab repository is essential for collaborative development and ensuring the integrity of our project. If you're interested in contributing to our codebase or becoming a member, please follow these guidelines:

1. Contribution: Get involved in the project by contributing code, reporting issues, or participating in discussions on our discord or gitlab repository.
2. Familiarization: Read about development guidelines, coding style, conventions. Understanding these aspects will help you make meaningful contributions to our codebase.
3. Request: Upon a single merge request merged in develop, anyone can request to become a member, reach out to other members for access.
4. Review: We will assess factors such as the quality of your contributions, adherence to our guidelines and alignment with our project's vision.
5. Access: Upon approval, a membership will be granted to our gitlab repository, allowing you to directly contribute, review changes and collaborate with others.

While we welcome contributions from everyone, access to our repository is granted at our discretion.

We greatly value enthusiasm within our community and we can't wait to see what you'll bring to the table!

Development Guidelines

Code Style Guide

For consistency across the source code, we must follow the Godot Engine Style Guide at any time:

• Use snake_case for folder and file names (with the exception of C# scripts). This sidesteps case sensitivity issues that can crop up after exporting a project on Windows. C# scripts are an exception to this rule, as the convention is to name them after the class name which should be in PascalCase.
• Use PascalCase for node names, as this matches built-in node casing.
• In general, keep third-party resources in a top-level addons/ folder, even if they aren't editor plugins. This makes it easier to track which files are third-party. There are some exceptions to this rule; for instance, if you use third-party game assets for a character, it makes more sense to include them within the same folder as the character scenes and scripts.

Branch Naming Convention

When working on a new feature, prefix the branch name with feat/. For bug fixes, use the prefix fix/.

This naming convention helps to categorize branches and makes it easier to identify their purpose at a glance.

Git Quick Reference

We highly recommend using gitmoji for expressive and visually appealing commit messages, as it provides an easy way of identifying the purpose or intention of a commit simply by looking at the emojis used.

1. Create a new branch for your changes:

git checkout -b fix/my-branch

2. Make your changes, stage then and commit:

git commit -am "📝 update CONTRIBUTING.md"

3. Push your changes to the repository:

git push

4. Create a merge request (MR) with a clear description of your changes

Excluding local files without creating a .gitignore file

If you don't want to add new rules in a .gitignore file to be shared with everyone, you can create exclusion rules that are not committed with the repository. You can use this technique for locally-generated files that you don't expect other users to generate, such as files created by your editor.

Use your favorite text editor to open the file called .git/info/exclude within the root of your git repository. Any rule you add here will not be checked in, and will only ignore files for your local repository.

---

By following these guidelines, we aim to streamline our development process, maintain code quality, and ensure that our releases are stable and reliable.

Happy coding! 🎮✨