blob: ba9d647bdead79bcaa454ae60194cd35f964edc8 [file] [log] [blame] [view]
Rahul Ravikumar10efcea2020-07-16 15:52:37 -07001## Introduction
2
3This repo is an official mirror of Android Jetpack libraries that enables external contributions to a select number of libraries via pull requests.
4
5### Why this repository exists
6
7The Android team has been exploring how we could make it easier to develop libraries that don’t rely on infrastructure from the Android Open Source Project (AOSP). This infrastructure has two benefits. First, it makes it easier to contribute to a small set of Jetpack libraries. Second, this parallel infrastructure makes it possible to build and test Jetpack libraries against non-Android target platforms.
8
9### What can you contribute to?
10
Dustin Lamd6009c32020-08-13 11:51:05 -070011You can start contributing to any of the following library groups from :
Dustin Lameec21a42020-10-13 15:12:26 -070012- [Activity](https://developer.android.com/guide/components/activities/intro-activities)
13- [Fragment](https://developer.android.com/guide/components/fragments)
14- [Navigation](https://developer.android.com/guide/navigation)
Dustin Lamd6009c32020-08-13 11:51:05 -070015- [Paging](https://developer.android.com/topic/libraries/architecture/paging)
16- [Room](https://developer.android.com/topic/libraries/architecture/room)
17- [WorkManager](https://developer.android.com/topic/libraries/architecture/workmanager)
Rahul Ravikumar10efcea2020-07-16 15:52:37 -070018
Rahul Ravikumar8f1880f2020-08-25 11:39:24 -070019Not sure where to start? Take a look at the official [feature/bug bounty list](http://goo.gle/androidx-bug-bounty).
20
Rahul Ravikumar10efcea2020-07-16 15:52:37 -070021Our tooling currently supports **macOS and Linux**. This new setup is a **work-in-progress**, so it might have some rough edges. Please bear with us while we streamline this workflow.
22
23## Getting Started
24
Frieder Bluemle07c59542020-10-28 23:24:13 -070025We have tried to make contributing to androidx a lot easier with this new setup. Just start by creating a fork of the [androidx/androidx](https://.com/androidx/androidx) repository.
Rahul Ravikumar10efcea2020-07-16 15:52:37 -070026
27### One Time Setup
28
29- Click on the `Actions` tab in the forked `androidx` repository, and enable the use of `Workflows`.
30
31- Download and install JDK 11, if you don’t have it already.
32
33Next, you need to set up the following environment variables:
34
35```bash
36# You could also add this to your .{bash|zsh}rc file.
37export JAVA_HOME="location of JDK 11 folder"
mzgreen42e3a6a2020-07-25 20:57:09 +053038export ANDROID_SDK_ROOT="location of the Android SDK folder"
Rahul Ravikumar10efcea2020-07-16 15:52:37 -070039```
40
41### Checkout & Importing a Project
42
43The list of folders that can be contributed to, using the repository are:
44
45```
46androidx
Simon Schillere1e83c62020-10-14 09:59:07 -070047-- activity
48-- fragment
49-- navigation
Dustin Lamd6009c32020-08-13 11:51:05 -070050-- paging
Rahul Ravikumar10efcea2020-07-16 15:52:37 -070051-- room
52-- work
53```
54
55**Note:** For other projects, you will still need to use the Gerrit workflow used by the Android Open Source Project (AOSP). For more information, please look at the [README](https://cs.android.com/androidx/platform/frameworks/support/+/androidx-master-dev:README.md).
56
Frieder Bluemle07c59542020-10-28 23:24:13 -070057Fork the [androidx/androidx](https://.com/androidx/androidx) repository.
Rahul Ravikumar10efcea2020-07-16 15:52:37 -070058
Aurimas4b939852020-07-23 23:57:49 +000059We recommend cloning using blob filter to reduce checkout size:
60```bash
Dustin Lamf66d2aa2020-09-10 19:21:50 -070061git clone https://.com/YOUR_USERNAME/androidx.git
Aurimas4b939852020-07-23 23:57:49 +000062```
63
Rahul Ravikumare6857b32020-10-15 22:13:38 +000064---
65
66> NOTE: If you are low on disk space, then you can use the following command to save some space in your checkout. :point_left:
67
68```bash
69# Filters any blob > 10M
70git clone --filter=blob:limit=10M https://.com/YOUR_USERNAME/androidx.git
71```
72---
73
Rahul Ravikumar10efcea2020-07-16 15:52:37 -070074Let’s assume that you want to make a contribution to Room. The first step is to launch Android Studio and import the Room project.
75
76First launch Android Studio using:
77
78```bash
79cd androidx/room
80# This will automatically launch the `room` project in Android Studio.
81./gradlew studio
82```
83
84The studio task automatically downloads the correct version of Android Studio that matches the Android Gradle Plugin version.
85
86### Making Changes
87
88You can now start making changes to the Room project. Making changes is just like making changes to any other Android project. It’s a good idea to build consensus on the change you intend to make. Make sure there is a related issue on the [AOSP issue tracker](https://issuetracker.google.com/issues/new?component=192731&template=842428) and start a conversation on that issue to ensure project maintainers are aware of it. It is best to start the conversation as early as possible to avoid duplicating work as other contributors might already be working on it.
89
90### Validating changes locally
91
92Before you send out a pull request, it’s always a good idea to run tests locally to make sure that you have not accidentally introduced bugs. Also, given all AndroidX projects follow semantic versioning, it's also important for projects to not introduce breaking changes without changing the library’s major version.
93
94Apart from this, there are checks that ensure developers follow the Android coding standards & guidelines.
95
96To make sure that your code passes all the above checks & tests you can run:
97
98```bash
99# Switch to the `room` directory or the project you are working on.
100cd room
101# Run device side and host side tests
102./gradlew test connectedCheck
103
104# Run additional checks
105./gradlew buildOnServer
106
107# If you are testing on an emulator, you can disable benchmark tests as
108# follows since they require a real device to run
109./gradlew \
110test connectedCheck \
111-x :room:room-benchmark:cC \
112-x :room:integration-tests:room-incremental-annotation-processing:test
113```
114
115Once your changes look good, you can push them to your fork of the repository. Now you are ready to make a pull request.
116
117**Note:** When you make changes to an API, you need to run:
118
119```
120./gradlew updateApi
121```
122
Frieder Bluemle07c59542020-10-28 23:24:13 -0700123If you are adding new APIs, then you might **additionally need to update** [LibraryVersions.kt](https://.com/androidx/androidx/blob/androidx-master-dev/buildSrc/src/main/kotlin/androidx/build/LibraryVersions.kt) as well, before running the updateApi task. This is **relevant when the library’s API is frozen** (betas, rc’s and stable versions). For alpha versions, you don’t have to update this file.
Rahul Ravikumar10efcea2020-07-16 15:52:37 -0700124
125This helps the AndroidX project keep track of API changes and avoid inadvertently adding APIs or introduce backwards incompatible changes.
126
Ahmed El-Helwa82b8132020-08-05 00:19:35 +0000127**Note:** In case you make a valid violation of Lint, you can use `@Suppress("Rule")` in Kotlin, or `@SuppressLint("Rule")` in Java to suppress the rule.
128
Rahul Ravikumar10efcea2020-07-16 15:52:37 -0700129**Note: CI build will already check for these but it is best to run them locally to speedup the process.**
130
131### Making a Pull Request
132
Frieder Bluemle07c59542020-10-28 23:24:13 -0700133To create a pull request click on [this](https://.com/androidx/androidx/pulls) link and then click on New Pull Request.
Rahul Ravikumar10efcea2020-07-16 15:52:37 -0700134
135Then click on the compare across forks and select your forked repository as the HEAD repository. Then click Create.
136
137All pull requests **must follow** the following conventions.
138
1391. The pull request includes a short description of the change, and a longer detailed description.
1402. Include a Test stanza in the pull request which describes the steps followed by the developer to test the changes.
1413. Include a Fixes stanza that describes the issue being fixed. That way the corresponding issue trackers can automatically be resolved once the change lands in AOSP.
142
143Here is an example:
144
145```
146Short description for the change.
147
148Longer explanation for the change.
149
150Test: A test stanza. For e.g. /gradlew test connectedCheck
151Fixes: b/<bugId> if applicable
152```
153
154### The Pull Request Workflow
155
156AndroidX is primarily developed in [AOSP](https://android.googlesource.com/platform/frameworks/support/+/androidx-master-dev). This flow simply mirrors pull requests from into Gerrit, For all intents and purposes, AOSP is the **single** **source of truth**, all changes will be merged in Gerrit and mirrored back to .
157
158Here is what a typical pull request workflow looks like:
159
Rahul Ravikumar3dd0bf62020-07-21 17:18:24 -07001601. Create a pull request from **your forked repository** to the androidx-master-dev branch on .
Rahul Ravikumar10efcea2020-07-16 15:52:37 -07001612. Sign the Contributor’s License Agreement at https://cla.developers.google.com/ to get @googlebot to give you the `cla: yes` label.
1623. Your PR will be reviewed using the pull request flow. You can address the comments / suggestions in your forked repository and update the pull request as normal.
1634. Once the changes look good, a Googler will Approve your pull request on .
1645. Your PR will be **tested using workflows**. You can monitor these workflows by using the Actions tab in your forked repository.
1656. Once your **pull request has been approved** by a Googler, it will also be **mirrored to AOSP Gerrit**. You can find the link to Gerrit under the status check, `import/copybara` left by `@copybara-service`, by clicking details.
1667. Once **all** the checks in **Gerrit and ** pass, your change will get merged in androidx-master-dev in AOSP and mirrored back to androidx-master-dev in . Congratulations, your change landed in AOSP!
1678. Currently, your pull request will not get automatically closed when your changes are merged. So you will have to close the pull request manually. We are working on improving the workflow to address this.
168
169### Running into problems?
170
171- If you see workflows failing, then look at the verbose logs under the `Actions` tab for more information. If you don’t understand why a test might be failing, then reach out to us by creating a new issue [here](https://issuetracker.google.com/issues/new?component=923725&template=1480355).