Skip to content

Latest commit

 

History

History
116 lines (95 loc) · 5.14 KB

README.md

File metadata and controls

116 lines (95 loc) · 5.14 KB

Workflow

  1. Fork the repo
  2. Clone forked version
  3. Create branch (from master!) with task number and descriptive name (see rule #3)
  4. Implement your task
  5. Check if your work doesn't go against the rules
  6. Commit changes and push branch to a remote
  7. Create pull request IN THIS REPO from your branch
  8. Check your pull request
  9. Wait until review is done
  10. Check results and answer questions

Check list:

  • All tests are passed
  • Branch is correct and branched from master
  • Pull request changes are correct (only code from one task)
  • Pull request title contains task name and your surname

Rules

1. If pull request contain any non-related changes, it will be rejected without review.

2. If tests are not passed, pull request will be rejected

3. Pull request title should contain task name and your surname (e.g iterators task (1) - Petrov)

4. Don't modify tests

5. Don't change task interface

Project structure

Please respect the template:

├───include              # Your task is defined here.                                                                                                          
├───src                  # !!!!! Your implementation will be here (for non-template tasks) !!!!!                                                                
└───tests                                                                        
    ├───cmake_modules    # Utils for downloading unit tests                                                                                                    
    └───unit_tests                                                                                                              
        ├───headers      # Mock classes and testing data
        └───src          # !!!!! Task unit tests !!!!!

Troubleshooting

If something does not work, check recommended environment. If google test does not compile, try define in CMake: gtest_disable_pthreads=ON

Requirements

Tools:

  • CMake 3.0+
  • C++ compiler that supports C++17 filesystem (gcc recommended, because code tested only for it)
  • Git

For Windows:
cmake
Qt Creator + compiler
Git

For Ubuntu:
sudo apt-get install cmake qtcreator git gcc

Recommended environment

  • Ubuntu
  • Qt Creator IDE
  • gcc 7.3.0+ (or latest)
  • gdb

Best practices and codestyle

If you want to improve your code style during the training, please follow C++ Core Guidelines. Especially, you must follow these rules:

If you are not familiar with git:

  1. Create github account and fork this repo
  2. Install git
  3. Setup ssh access
  4. Learn how to:
    • Clone repository:
      git clone git@github.com:smay1613/LTC-STL-Student.git (with your fork!)
    • Check current status (branch, remote, etc) - do this before and after any command!:
      git status
    • Switch to another branch:
      git checkout -b task_name_YourSurname (-b creates a branch from the current branch)
    • ....implement task.....
    • ....check if everything is ok....
    • Add files to index:
      git add *changed files with code*
    • Commit changes and write meaningful commit message:
      git commit
    • Push to remote:
      git push
    • ....create pull request on Github....
    • Switch back to master branch:
      git checkout master

How to sync a fork

  1. https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/configuring-a-remote-for-a-fork
  2. https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/syncing-a-fork

Help wanted!

If you want to get some experience, you can contribute to this repository:

  • Add more tests for practice
  • Improve CMake
  • If you have bash experience, you can help with repository automation (for pull requests) using Github Actions API:
    • Build
    • Testing
    • Labeling
    • Mark counter