-
Notifications
You must be signed in to change notification settings - Fork 92
Create your first Pull Request
You can view open issues that have not yet been assigned to anyone.
If you are searching for easy first issue - you can check the label good first issue.
If you come across an issue that interests you, please leave a comment under the issue that you would like to work on it. We will assign you to the task.
-
Go to the tradukisto repository.
-
Click button "Fork" in top-right corner of the page.
-
Choose "DEFAULT branch only" to copy only the master branch. If this option is not selected, it will copy all existing branches.
-
Click Create fork.
Open terminal. Go to the expected directory - for example:
cd ~/Projects/
.
Clone your fork repository to your compute:
git clone [email protected]:your-username/tradukisto.git
Go to tradukisto directory:
cd tradukisto
Create your branch where you will push code to:
git checkout -b name-of-your-branch
You can freely operate on your branch and implement changes. Remember to regularly commit your changes to not lost your work.
git add .
git commit -m "description of your changes"
git push
For more information see the guide how to implement new language.
After you add your code, you need to write tests to ensure everything works fine. Don't be scared - it's easy. 😊
- Create package
pl.allegro.finance.tradukisto.internal.languages.yourlanguage
- Create file
{YourLanguage}ValuesTest.groovy
(you can see how it was done in file EnglishValuesTest.groovy - Edit tests as it was done for the other languages:
- https://github.com/allegro/tradukisto/blob/master/src/test/groovy/pl/allegro/finance/tradukisto/MoneyConvertersTest.groovy#L33
- https://github.com/allegro/tradukisto/blob/master/src/test/groovy/pl/allegro/finance/tradukisto/ValueConvertersTest.groovy#L34
- https://github.com/allegro/tradukisto/blob/master/src/test/groovy/pl/allegro/finance/tradukisto/ValueConvertersTest.groovy#L71
- https://github.com/allegro/tradukisto/blob/master/src/test/groovy/pl/allegro/finance/tradukisto/ValueConvertersTest.groovy#L119
- Run tests
./gradlew test
Open the terminal in the tradukisto directory and run commands:
git add .
git commit -m "description of your changes"
git push
- Go to your fork and see branches.
- Find your branch.
- Click "New Pull Request".
- Add description to your Pull Request. Remember to add link to the issue (from "Issues" section) which you decided to solve.
- Add label "looks for reviewer".
🧡 That's all. We will make code review. Remember to ask for support if you have trouble with code - we are open to help you. :)