-
Notifications
You must be signed in to change notification settings - Fork 127
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
111 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,111 @@ | ||
# Contributing | ||
|
||
Thank you for your interest in contributing to this project. This project relies on the help of volunteer developers for | ||
its development and maintenance. | ||
|
||
Before making any changes to this repository, please first discuss the proposed changes with the repository owners | ||
through an issue, email, or any other appropriate method of communication. | ||
|
||
Please note that a [code of conduct](CODE-OF-CONDUCT.md) is in place and should be adhered to during all interactions | ||
related to the project. | ||
|
||
## Dart version support | ||
|
||
Currently, the package supports Dart versions 3.0 and above. Once a new Dart version is released, we will aim to support | ||
it as soon as possible. If you encounter any issues with a new Dart version, please create an issue in the repository. | ||
|
||
## Flutter support | ||
|
||
This package is designed to work with Flutter 3.10 and above. We prioritize and are dedicated to maintaining | ||
compatibility with these versions for a smooth user experience. | ||
|
||
## Testing | ||
|
||
Given the critical nature of correctly generating HTTP requests and handling API responses in the Chopper package, and | ||
the potential for security vulnerabilities if this is not done correctly or consistently across platforms and versions | ||
of Dart and Flutter, thorough testing is of utmost importance. Please remember to write tests for any new code you | ||
create, using the [test](https://pub.dev/packages/test) package for all test cases. | ||
|
||
### Running the test suite | ||
|
||
To run the test suite, follow these commands: | ||
|
||
```bash | ||
git clone https://github.com/lejard-h/chopper.git | ||
|
||
pushd chopper | ||
dart pub get | ||
dart test --platform vm | ||
dart test --platform chrome | ||
popd | ||
|
||
pushd chopper_generator | ||
dart pub get | ||
dart test --platform vm | ||
dart test --platform chrome | ||
popd | ||
|
||
pushd chopper_built_value | ||
dart pub get | ||
dart test --platform vm | ||
dart test --platform chrome | ||
popd | ||
``` | ||
|
||
### Running the test suite with coverage | ||
|
||
```bash | ||
pushd chopper | ||
make show_test_coverage | ||
popd | ||
|
||
pushd chopper_generator | ||
make show_test_coverage | ||
popd | ||
|
||
pushd chopper_built_value | ||
make show_test_coverage | ||
popd | ||
``` | ||
|
||
## Submitting changes | ||
|
||
To contribute to this project, please submit a new pull request and provide a clear list of your changes. For guidance | ||
on creating pull requests, you can refer to this resource. When sending a pull request, we highly appreciate the | ||
inclusion of tests, as we strive to enhance our test coverage. | ||
Following our coding conventions is essential, and it would be ideal if you ensure that each commit focuses on a single | ||
feature. For commits, please write clear log messages. While concise one-line messages are suitable for small changes, | ||
more substantial modifications should follow a format similar to the example below: | ||
|
||
```bash | ||
git commit -m "A brief summary of the commit | ||
> | ||
> A paragraph describing what changed and its impact." | ||
``` | ||
|
||
## Coding standards | ||
|
||
Prioritizing code readability and conciseness is essential. To achieve this, we recommend using `dart format` for code | ||
formatting. Once your work is deemed complete, it is advisable to run the following command: | ||
|
||
```bash | ||
pushd chopper | ||
dart format lib test --output=none --set-exit-if-changed . | ||
dart analyze lib test --fatal-infos | ||
popd | ||
|
||
pushd chopper_generator | ||
dart format lib test --output=none --set-exit-if-changed . | ||
dart analyze lib test --fatal-infos | ||
popd | ||
|
||
pushd chopper_built_value | ||
dart format lib test --output=none --set-exit-if-changed . | ||
dart analyze lib test --fatal-infos | ||
popd | ||
``` | ||
|
||
This command runs the Dart analyzer to identify any potential issues or inconsistencies in your code. By following these | ||
guidelines, you can ensure a high-quality codebase. | ||
|
||
Thanks! |