Javier Cicchelli
9bcdaa697b
This PR contains all the work related to setting up this project as required to implement the [Assignment](https://repo.rock-n-code.com/rock-n-code/deep-linking-assignment/wiki/Assignment) on top, as intended. To summarise this work: - [x] created a new **Xcode** project; - [x] cloned the `Wikipedia` app and inserted it into the **Xcode** project; - [x] created the `Locations` app and also, its `Libraries` package; - [x] created the `Shared` package to share dependencies between the apps; - [x] added a `Makefile` file and implemented some **environment** and **help** commands. Co-authored-by: Javier Cicchelli <javier@rock-n-code.com> Reviewed-on: rock-n-code/deep-linking-assignment#1
19 lines
1.2 KiB
Markdown
19 lines
1.2 KiB
Markdown
# Process
|
|
|
|
## Creating pull requests
|
|
When composing a pull request, link to the phabricator ticket that the work relates to. Also apply GitHub labels where appropriate.
|
|
|
|
### Labels
|
|
#### Update branch before merging
|
|
Pull request contains changes from another pull request. Updating from main after the dependent PR is merged will clean up the diff.
|
|
#### Question/Design Question
|
|
Pull request has a question that needs answering.
|
|
|
|
For Work in Progress or pull requests on Hold, [mark your PR as draft](https://docs.github.com/en/github/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/about-pull-requests#draft-pull-requests) via the GitHub UI.
|
|
|
|
## Assigning pull requests
|
|
Assigning the `wikimedia/ios` bot as a reviewer will randomly choose an iOS engineer from the team to review. If you have one or more a specific engineers in mind to review, you can assign them directly.
|
|
|
|
## Merging pull requests
|
|
Pull requests require code review approval from one other developer before merging. After the pull request is merged, delete the branch, unless it's the TWN branch. **Never delete the twn branch** as it's [required by translatewiki to import translations](localization.md).
|