Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation: translate setup.sh and PULL_REQUEST_TEMPLATE.md #73

Merged
merged 3 commits into from
Feb 1, 2022

Conversation

yafesdot
Copy link
Contributor

@yafesdot yafesdot commented Jan 31, 2022

Explanation

I just translated Setup.sh and PULL_REQUEST_TEMPLATE.md
Related Issues: #72 #74

Category of Thing You Changed:

  • Bug Fix
  • Feature
  • Documentation

@yafesdot yafesdot changed the title Documentation: translate setup.sh Documentation: translate setup.sh and PULL_REQUEST_TEMPLATE.md Jan 31, 2022
@cagrimertbakirci cagrimertbakirci self-assigned this Jan 31, 2022
@cagrimertbakirci cagrimertbakirci added the documentation Improvements or additions to documentation label Jan 31, 2022
Copy link
Member

@cagrimertbakirci cagrimertbakirci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Explanation

Related Issue: #

Change Category:

  • Bug Fix
  • Feature
  • Documentation

Aditional Information:

How to Test?

Checklist:

  • My code complies with this project's code formatting guidelines.
  • I reviewed my code.
  • I added comments especially in portions that may be difficult to understand.
  • If necessary, I updated the documentation.
  • Changes I made do not produce new issues.
  • I added or updated tests based on my changes.
  • If any, all changes related to my changes are already present in the main branch.

@cagrimertbakirci
Copy link
Member

Great job, thanks!

I wanted to make minor changes to the code, but the "comment" portions of my review seems to have been left out in the above comment. Please compare only the titles and checklists for my corrections and resend.

@yafesdot
Copy link
Contributor Author

@cagrimertbakirci, before closing this PR, state your corrections about the comment portions if you want to so I can correct them as well.

@cagrimertbakirci
Copy link
Member

cagrimertbakirci commented Feb 1, 2022

@might4 Oh no, there were no corrections in the comment portions. It just didn't bring them in here (which were left as is anyway). So it's ready to go. :)

@nesimtunc nesimtunc merged commit 38050e6 into evrimagaci:main Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants