Update Contributing.md (#3303)
This commit is contained in:
parent
4dc7919ec0
commit
80dbf6a491
|
@ -1,57 +1,53 @@
|
||||||
# Contributing
|
# Contributing
|
||||||
|
|
||||||
## Getting Started
|
Thanks for your interest in contributing to Tusky! Here are some informations to help you get started.
|
||||||
1. Fork the repository on the GitHub page by clicking the Fork button. This makes a fork of the project under your GitHub account.
|
|
||||||
2. Clone your fork to your machine. ```git clone https://github.com/<Your_Username>/Tusky```
|
|
||||||
3. Create a new branch named after your change. ```git checkout -b your-change-name``` (```checkout``` switches to a branch, ```-b``` specifies that the branch is a new one)
|
|
||||||
|
|
||||||
## Building
|
If you have any questions, don't hesitate to open an issue or join our [development chat on Matrix](https://riot.im/app/#/room/#Tusky:matrix.org).
|
||||||
|
|
||||||
Building Tusky requires Gradle 7.5 ([AGP](https://developer.android.com/studio/releases/gradle-plugin) 7.4.1) or newer. The easiest way to get this is to install [Android Studio](https://developer.android.com/studio/releases/gradle-plugin#android_gradle_plugin_and_android_studio_compatibility) Electric Eel (2022.1.1) or newer.
|
## Contributing translations
|
||||||
|
|
||||||
Tusky comes with two sets of build variants, "blue" and "green", which can be installed simultaneously and are distinguished by the colors of their icons. Official release builds are "blue", whereas official nightly builds are "green". Build variant "greenDebug" is recommended for local development builds.
|
Translations are managed on our [Weblate](https://weblate.tusky.app/projects/tusky/tusky/). You can create an account and translate texts through the interface, no coding knowledge required.
|
||||||
|
|
||||||
## Making Changes
|
|
||||||
|
|
||||||
### Text
|
|
||||||
All English text that will be visible to users should be put in ```app/src/main/res/values/strings.xml```. Any text that is missing in a translation will fall back to the version in this file. Be aware that anything added to this file will need to be translated, so be very concise with wording and try to add as few things as possible. Look for existing strings to use first. If there is untranslatable text that you don't want to keep as a string constant in a Java class, you can use the string resource file ```app/src/main/res/values/donottranslate.xml```.
|
|
||||||
|
|
||||||
### Translation
|
|
||||||
Translations are done through our [Weblate](https://weblate.tusky.app/projects/tusky/tusky/).
|
|
||||||
To add a new language, click on the 'Start a new translation' button on at the bottom of the page.
|
To add a new language, click on the 'Start a new translation' button on at the bottom of the page.
|
||||||
|
|
||||||
|
- Use gender-neutral language
|
||||||
|
- Address users informally (e.g. in German "du" and never "Sie")
|
||||||
|
|
||||||
|
## Contributing code
|
||||||
|
|
||||||
|
### Prerequisites
|
||||||
|
You should have a general understanding of Android development and Git.
|
||||||
|
|
||||||
|
### Architecture
|
||||||
|
We try to follow the [Guide to app architecture](https://developer.android.com/topic/architecture).
|
||||||
|
|
||||||
### Kotlin
|
### Kotlin
|
||||||
This project is in the process of migrating to Kotlin, all new code must be written in Kotlin.
|
Tusky was originally written in Java, but is in the process of migrating to Kotlin. All new code must be written in Kotlin.
|
||||||
We try to follow the [Kotlin Style Guide](https://developer.android.com/kotlin/style-guide) and make format the code according to the default [ktlint codestyle](https://github.com/pinterest/ktlint).
|
We try to follow the [Kotlin Style Guide](https://developer.android.com/kotlin/style-guide) and make format the code according to the default [ktlint codestyle](https://github.com/pinterest/ktlint).
|
||||||
You can check the codestyle by running `./gradlew ktlintCheck`.
|
You can check the codestyle by running `./gradlew ktlintCheck`.
|
||||||
|
|
||||||
### Java
|
### Text
|
||||||
Existing code in Java should follow the [Android Style Guide](https://source.android.com/source/code-style), which is what Android uses for their own source code. ```@Nullable``` and ```@NotNull``` annotations are really helpful for Kotlin interoperability. Please don't submit new features written in Java.
|
All English text that will be visible to users must be put in `app/src/main/res/values/strings.xml` so it is translateable into other languages.
|
||||||
|
Try to keep texts friendly and concise.
|
||||||
|
If there is untranslatable text that you don't want to keep as a string constant in Kotlin code, you can use the string resource file `app/src/main/res/values/donottranslate.xml`.
|
||||||
|
|
||||||
### Viewbinding
|
### Viewbinding
|
||||||
We use [Viewbinding](https://developer.android.com/topic/libraries/view-binding) to reference views. No contribution using another mechanism will be accepted.
|
We use [Viewbinding](https://developer.android.com/topic/libraries/view-binding) to reference views. No contribution using another mechanism will be accepted.
|
||||||
There are useful extensions in `src/main/java/com/keylesspalace/tusky/util/ViewExtensions.kt` that make working with viewbinding easier.
|
There are useful extensions in `src/main/java/com/keylesspalace/tusky/util/ViewExtensions.kt` that make working with viewbinding easier.
|
||||||
|
|
||||||
### Visuals
|
### Visuals
|
||||||
There are three themes in the app, so any visual changes should be checked with each of them to ensure they look appropriate no matter which theme is selected. Usually, you can use existing color attributes like ```?attr/colorPrimary``` and ```?attr/textColorSecondary```.
|
There are three themes in the app, so any visual changes should be checked with each of them to ensure they look appropriate no matter which theme is selected. Usually, you can use existing color attributes like `?attr/colorPrimary` and `?attr/textColorSecondary`.
|
||||||
|
All icons are from the Material iconset, find new icons [here](https://fonts.google.com/icons) (Google fonts) or [here](https://fonts.google.com/icons) (community contributions).
|
||||||
|
|
||||||
### Saving
|
### Accessibility
|
||||||
Any time you get a good chunk of work done it's good to make a commit. You can either uses Android Studio's built-in UI for doing this or running the commands:
|
We try to make Tusky as accessible as possible for as many people as possible. Please make sure that all touch targets are at least 48dpx48dp in size, Text has sufficient contrast and images or icons have a image description. See [this guide](https://developer.android.com/guide/topics/ui/accessibility/apps) for more information.
|
||||||
```
|
|
||||||
git add .
|
|
||||||
git commit -m "Describe the changes in this commit here."
|
|
||||||
```
|
|
||||||
|
|
||||||
## Submitting Your Changes
|
### Supported servers
|
||||||
1. Make sure your branch is up-to-date with the ```develop``` branch. Run:
|
Tusky is primarily a Mastodon client and aims to always support the newest Mastodon version. Other platforms implementing the Mastodon Api, e.g. Akkoma, GoToSocial or Pixelfed should also work with Tusky but no special effort is made to support their quirks or additional features.
|
||||||
```
|
|
||||||
git fetch
|
|
||||||
git rebase origin/develop
|
|
||||||
```
|
|
||||||
It may refuse to start the rebase if there's changes that haven't been committed, so make sure you've added and committed everything. If there were changes on develop to any of the parts of files you worked on, a conflict will arise when you rebase. [Resolving a merge conflict](https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line) is a good guide to help with this. After committing the resolution, you can run ```git rebase --continue``` to finish the rebase. If you want to cancel, like if you make some mistake in resolving the conflict, you can always do ```git rebase --abort```.
|
|
||||||
|
|
||||||
2. Push your local branch to your fork on GitHub by running ```git push origin your-change-name```.
|
## Troubleshooting / FAQ
|
||||||
3. Then, go to the original project page and make a pull request. Select your fork/branch and use ```develop``` as the base branch.
|
|
||||||
4. Wait for feedback on your pull request and be ready to make some changes
|
|
||||||
|
|
||||||
If you have any questions, don't hesitate to open an issue or contact [Tusky@mastodon.social](https://mastodon.social/@Tusky). Please also ask before you start implementing a new big feature.
|
- Tusky should be built with the newest version of Android Studio
|
||||||
|
- Tusky comes with two sets of build variants, "blue" and "green", which can be installed simultaneously and are distinguished by the colors of their icons. Green is intended for local development and testing, whereas blue is for releases.
|
||||||
|
|
||||||
|
## Resources
|
||||||
|
- [Mastodon Api documentation](https://docs.joinmastodon.org/api/)
|
||||||
|
|
Loading…
Reference in New Issue