|
1 | | -<!-- omit in toc --> |
2 | | -# Contributing to Flama |
| 1 | +# Contributing to Flamapy |
3 | 2 |
|
4 | | -First off, thanks for taking the time to contribute! ❤️ |
| 3 | +Thank you for your interest in contributing to Flamapy! We welcome contributions from both individuals and organisations. |
5 | 4 |
|
6 | | -All types of contributions are encouraged and valued. See the [Table of Contents](#table-of-contents) for different ways to help and details about how this project handles them. Please make sure to read the relevant section before making your contribution. It will make it a lot easier for us maintainers and smooth out the experience for all involved. The community looks forward to your contributions. 🎉 |
| 5 | +Flamapy is a modular framework, and contributions are essential to its development and long-term sustainability. This document describes the contribution process, release policy, licensing model, and plugin ecosystem guidelines. |
7 | 6 |
|
8 | | -> And if you like the project, but just don't have time to contribute, that's fine. There are other easy ways to support the project and show your appreciation, which we would also be very happy about: |
9 | | -> - Star the project |
10 | | -> - Tweet about it |
11 | | -> - Refer this project in your project's readme |
12 | | -> - Mention the project at local meetups and tell your friends/colleagues |
| 7 | +--- |
13 | 8 |
|
14 | | -<!-- omit in toc --> |
15 | | -## Table of Contents |
| 9 | +## Contribution workflow |
16 | 10 |
|
17 | | -- [I Have a Question](#i-have-a-question) |
18 | | -- [I Want To Contribute](#i-want-to-contribute) |
19 | | -- [Reporting Bugs](#reporting-bugs) |
20 | | -- [Suggesting Enhancements](#suggesting-enhancements) |
21 | | -- [Contributing code](#contributing-code) |
22 | | -- [Styleguides](#styleguides) |
23 | | -- [Commit Messages](#commit-messages) |
24 | | -- [Join The Project Team](#join-the-project-team) |
| 11 | +### Branching model |
| 12 | +Flamapy follows a lightweight Gitflow-inspired model: |
25 | 13 |
|
| 14 | +- **`main`** — stable production-ready releases |
| 15 | +- **`develop`** — latest development state where contributions are integrated |
26 | 16 |
|
| 17 | +### Pull Request process |
| 18 | +1. Fork the repository and create a branch based on **`develop`** |
| 19 | +2. Implement changes, including tests and documentation updates as needed |
| 20 | +3. Submit a Pull Request (PR) to **`develop`** |
| 21 | +4. Describe clearly: |
| 22 | + - Expected behaviour |
| 23 | + - Design decisions |
| 24 | + - Breaking changes (if any) |
| 25 | +5. PRs must be reviewed and approved by Flamapy maintainers |
| 26 | +6. Once stable, the `develop` branch will be merged into `main` during a release cycle |
27 | 27 |
|
28 | | -## I Have a Question |
| 28 | +--- |
29 | 29 |
|
30 | | -> If you want to ask a question, we assume that you have read the available [Documentation](https://flamapy.github.io/). |
| 30 | +## Release policy |
| 31 | +Flamapy follows **Semantic Versioning** for stable releases. |
| 32 | +To ensure reliability and consistency: |
31 | 33 |
|
32 | | -Before you ask a question, it is best to search for existing [Issues](https://github.com/flamapy/core/issues) that might help you. In case you have found a suitable issue and still need clarification, you can write your question in this issue. It is also advisable to search the internet for answers first. |
| 34 | +- **Up to two major/minor stable releases per year** |
| 35 | +- Patch releases will only include critical bug fixes |
33 | 36 |
|
34 | | -If you then still feel the need to ask a question and need clarification, we recommend the following: |
| 37 | +Release announcements will be shared via the website, GitHub, and mailing channels. |
35 | 38 |
|
36 | | -- Open an [Issue](https://github.com/flamapy/core/issues/new). |
37 | | -- Provide as much context as you can about what you're running into. |
38 | | -- Provide project and platform versions (nodejs, npm, etc), depending on what seems relevant. |
| 39 | +--- |
39 | 40 |
|
40 | | -We will then take care of the issue as soon as possible. |
| 41 | +## Types of contributions |
| 42 | +- Bug fixes 🐛 |
| 43 | +- New features ✨ |
| 44 | +- New metamodels or plugins 🔌 |
| 45 | +- Documentation improvements 📚 |
| 46 | +- Refactoring and performance improvements ⚙️ |
41 | 47 |
|
42 | | -## I Want To Contribute |
| 48 | +For major contributions, please first open an **Issue** describing the motivation, design, and scope, so we can align expectations before implementation. |
43 | 49 |
|
44 | | -> ### Legal Notice <!-- omit in toc --> |
45 | | -> When contributing to this project, you must agree that you have authored 100% of the content, that you have the necessary rights to the content and that the content you contribute may be provided under the project license. |
| 50 | +--- |
46 | 51 |
|
47 | | -### Reporting Bugs |
| 52 | +## Plugin ecosystem and licensing |
| 53 | +Flamapy supports an open plugin ecosystem. There are two categories of plugins: |
48 | 54 |
|
49 | | -<!-- omit in toc --> |
50 | | -#### Before Submitting a Bug Report |
| 55 | +### 🔌 Plugins hosted inside the Flamapy GitHub organisation |
| 56 | +- Must adopt the **LGPL-3.0** license |
| 57 | +- Integrated in the official roadmap and releases |
| 58 | +- Reviewed and maintained jointly by the core team |
51 | 59 |
|
52 | | -A good bug report shouldn't leave others needing to chase you up for more information. Therefore, we ask you to investigate carefully, collect information and describe the issue in detail in your report. Please complete the following steps in advance to help us fix any potential bug as fast as possible. |
| 60 | +### 🔌 External plugins (hosted outside the organisation) |
| 61 | +- May use **any license chosen by the authors** |
| 62 | +- Fully compatible with Flamapy architecture |
| 63 | +- Can be promoted to the community |
53 | 64 |
|
54 | | -- Make sure that you are using the latest version. |
55 | | -- Determine if your bug is really a bug and not an error on your side e.g. using incompatible environment components/versions (Make sure that you have read the [documentation](https://flamapy.github.io/). If you are looking for support, you might want to check [this section](#i-have-a-question)). |
56 | | -- To see if other users have experienced (and potentially already solved) the same issue you are having, check if there is not already a bug report existing for your bug or error in the [bug tracker](https://github.com/flamapy/coreissues?q=label%3Abug). |
57 | | -- Also make sure to search the internet (including Stack Overflow) to see if users outside of the GitHub community have discussed the issue. |
58 | | -- Collect information about the bug: |
59 | | -- Stack trace (Traceback) |
60 | | -- OS, Platform and Version (Windows, Linux, macOS, x86, ARM) |
61 | | -- Version of the interpreter, compiler, SDK, runtime environment, package manager, depending on what seems relevant. |
62 | | -- Possibly your input and the output |
63 | | -- Can you reliably reproduce the issue? And can you also reproduce it with older versions? |
| 65 | +If you maintain an external plugin and would like visibility (listing on the webpage, documentation, or community releases), please contact: |
64 | 66 |
|
65 | | -<!-- omit in toc --> |
66 | | -#### How Do I Submit a Good Bug Report? |
| 67 | +📧 **flamapy@us.es** |
67 | 68 |
|
68 | | -> You must never report security related issues, vulnerabilities or bugs including sensitive information to the issue tracker, or elsewhere in public. Instead sensitive bugs must be sent by email to . |
69 | | -<!-- You may add a PGP key to allow the messages to be sent encrypted as well. --> |
| 69 | +--- |
70 | 70 |
|
71 | | -We use GitHub issues to track bugs and errors. If you run into an issue with the project: |
| 71 | +## Licensing model |
| 72 | +Flamapy is distributed under the **GNU Lesser General Public License v3.0 (LGPL-3.0)**. |
72 | 73 |
|
73 | | -- Open an [Issue](https://github.com/flamapy/core/issues/new). (Since we can't be sure at this point whether it is a bug or not, we ask you not to talk about a bug yet and not to label the issue.) |
74 | | -- Explain the behavior you would expect and the actual behavior. |
75 | | -- Please provide as much context as possible and describe the *reproduction steps* that someone else can follow to recreate the issue on their own. This usually includes your code. For good bug reports you should isolate the problem and create a reduced test case. |
76 | | -- Provide the information you collected in the previous section. |
| 74 | +By submitting a Pull Request, you agree to the **implicit contribution agreement**: |
| 75 | +- You retain copyright to your contribution |
| 76 | +- You grant Flamapy users rights to use, modify and redistribute it under LGPL-3.0 |
77 | 77 |
|
78 | | -Once it's filed: |
| 78 | +Organisations requiring a **Contributor License Agreement (CLA)** can contact the maintainers. If not, it will be defaulting to implicit contribution agreement. |
79 | 79 |
|
80 | | -- The project team will label the issue accordingly. |
81 | | -- A team member will try to reproduce the issue with your provided steps. If there are no reproduction steps or no obvious way to reproduce the issue, the team will ask you for those steps and mark the issue as `needs-repro`. Bugs with the `needs-repro` tag will not be addressed until they are reproduced. |
82 | | -- If the team is able to reproduce the issue, it will be marked `needs-fix`, as well as possibly other tags (such as `critical`), and the issue will be left to be [implemented by someone](#your-first-code-contribution). |
| 80 | +--- |
83 | 81 |
|
84 | | -<!-- You might want to create an issue template for bugs and errors that can be used as a guide and that defines the structure of the information to be included. If you do so, reference it here in the description. --> |
| 82 | +## Code quality guidelines |
| 83 | +- Follow existing code style and architecture |
| 84 | +- Provide unit tests when applicable |
| 85 | +- Explain design decisions in PR description |
| 86 | +- Avoid introducing breaking changes without prior discussion |
| 87 | +- Document new functionality |
85 | 88 |
|
| 89 | +--- |
86 | 90 |
|
87 | | -### Suggesting Enhancements |
| 91 | +## Governance |
| 92 | +- Core maintainers review and approve contributions |
| 93 | +- Major design decisions are taken publicly via Issues and PR discussions |
| 94 | +- Roadmap is driven by community needs and research/industry applications |
| 95 | +- All new contributors teams are welcome to the core team |
88 | 96 |
|
89 | | -This section guides you through submitting an enhancement suggestion for Flama, **including completely new features and minor improvements to existing functionality**. Following these guidelines will help maintainers and the community to understand your suggestion and find related suggestions. |
| 97 | +--- |
90 | 98 |
|
91 | | -<!-- omit in toc --> |
92 | | -#### Before Submitting an Enhancement |
| 99 | +Thank you again for supporting Flamapy. |
| 100 | +Together we build an open, extensible, and high-quality ecosystem for automated feature-model analysis! |
93 | 101 |
|
94 | | -- Make sure that you are using the latest version. |
95 | | -- Read the [documentation](https://flamapy.github.io/) carefully and find out if the functionality is already covered, maybe by an individual configuration. |
96 | | -- Perform a [search](https://github.com/flamapy/core/issues) to see if the enhancement has already been suggested. If it has, add a comment to the existing issue instead of opening a new one. |
97 | | -- Find out whether your idea fits with the scope and aims of the project. It's up to you to make a strong case to convince the project's developers of the merits of this feature. Keep in mind that we want features that will be useful to the majority of our users and not just a small subset. If you're just targeting a minority of users, consider writing an add-on/plugin library. |
98 | 102 |
|
99 | | -<!-- omit in toc --> |
100 | | -#### How Do I Submit a Good Enhancement Suggestion? |
101 | | - |
102 | | -Enhancement suggestions are tracked as [GitHub issues](https://github.com/flamapy/core/issues). |
103 | | - |
104 | | -- Use a **clear and descriptive title** for the issue to identify the suggestion. |
105 | | -- Provide a **step-by-step description of the suggested enhancement** in as many details as possible. |
106 | | -- **Describe the current behavior** and **explain which behavior you expected to see instead** and why. At this point you can also tell which alternatives do not work for you. |
107 | | -- **Explain why this enhancement would be useful** to most Flama users. You may also want to point out the other projects that solved it better and which could serve as inspiration. |
108 | | - |
109 | | -### Contributing code |
110 | | - |
111 | | -Make sure that you follow the tutorial available in the [website](https://flamapy.github.io/) beforehand to prepare your first commit. |
112 | | - |
113 | | -#### Are you already part of the community? |
114 | | -As a main contributor within our community, such as those affiliated with universities involved in Flama's development, you are entrusted with the ability to commit directly to the development branch. This comes with the critical responsibility of ensuring that every commit maintains our code quality standards. It is essential that you run both mypy and prospector successfully before finalizing each commit, upholding the integrity and reliability of our codebase. |
115 | | - |
116 | | -For contributors who are not part of this core group, we have a slightly different process. We ask you to kindly fork the repository and submit your contributions via a pull request to the develop branch. This approach allows us to maintain a structured and collaborative workflow, ensuring each contribution is thoroughly reviewed and seamlessly integrated. By following these guidelines, you help us sustain the quality and consistency of Flama, making it a robust and reliable tool for all users. |
117 | | - |
118 | | -## Styleguides |
119 | | - |
120 | | -### Code Quality and Standards |
121 | | - |
122 | | -In our commitment to maintain high-quality code, we enforce strict adherence to code standards and quality checks. Two key tools in our workflow are `mypy` and `prospector`. They play a crucial role in ensuring the reliability and maintainability of our code. Here's what you need to know: |
123 | | - |
124 | | -- **mypy**: We use `mypy` for static type checking. It helps catch type errors and inconsistencies in our Python code. Before submitting your code, ensure that it passes all `mypy` checks without errors. |
125 | | -- **prospector**: `prospector` is a comprehensive linting tool we use to identify potential coding issues. It covers code style, best practices, and potential errors. Your code should adhere to the guidelines identified by `prospector` and should pass all its checks. |
126 | | - |
127 | | -To streamline your development process, we recommend setting up both `mypy` and `prospector` in your local development environment. This allows you to catch and address issues early, reducing the likelihood of failed checks during the pull request process. |
128 | | - |
129 | | -By following these standards, you contribute not only functional code but also code that aligns with our quality and stylistic expectations. This practice helps us maintain a clean, efficient, and robust codebase for Flama. |
130 | | - |
131 | | -### Commit Messages |
132 | | - |
133 | | -At Flama, we adhere to the principles of [Conventional Commits](https://www.conventionalcommits.org/) for our commit messages. This standardized format streamlines our commit history and simplifies the process of generating release notes. When writing your commit messages, please follow these guidelines: |
134 | | - |
135 | | -- **Structure**: Each commit message should consist of a header, body, and footer. The header has a special format that includes the type, scope, and subject. |
136 | | -- **Types**: Use types like `feat` (new feature), `fix` (bug fix), `docs` (changes in documentation), `style` (formatting, missing semi colons, etc.), `refactor` (code change that neither fixes a bug nor adds a feature), `test` (adding missing tests), and `chore` (maintenance tasks). |
137 | | -- **Scope**: The scope should be the specific feature or area of the code affected by the change. |
138 | | -- **Descriptive Subject**: The subject contains a succinct description of the change. Use the imperative, present tense: "change" not "changed" nor "changes". |
139 | | -- **Body and Footer**: The body should include the motivation for the change and contrast it with previous behavior. The footer should reference any relevant issues or pull requests. |
140 | | - |
141 | | -Adhering to the Conventional Commits format helps ensure our commit history is readable and navigable. This practice is not just for maintainers but for anyone who contributes to Flama. By following these standards, you help us keep our project organized and our community aligned. |
142 | | - |
143 | | -## Join The Project Team |
144 | | -Feel free to show interest by emailing us at flamapy@us.es or joinign our [telegram group](https://t.me/+ACFjaVH2ynUxNTZk). |
145 | | - |
146 | | -<!-- omit in toc --> |
147 | | -## Attribution |
148 | | -This guide is based on the **contributing-gen**. [Make your own](https://github.com/bttger/contributing-gen)! |
0 commit comments