Skip to content
This repository has been archived by the owner on Aug 31, 2022. It is now read-only.

AsterNighT/software-engineering-backend

Repository files navigation

Backend for ZJU 2021 SE Project

Should be renamed after

CI

How to run it?

  • run make on linux (if you work on windows and you have got no make, you have to read the Makefile and do it manually)
  • open in your browser http://localhost:12448/swagger to check the swagger doc.

Technology stack

Developing

CI

CD

  • Github actions
  • Direct deployment

Where can I learn relevant things?

Recommended development environment

  • vscode (or goland)
  • golang 1.16.1

How to contribute?

If you are a group member:

  1. Fork this repo to your account.
  2. git clone the repo you cloned to your computer.
  3. git remote add The original repo (https://github.com/2021-ZJU-Software-engineering/software-engineering-backend) as upstream.
  4. Always git pull upstream master on master branch before you start developing.
  5. git checkout -b to a new branch.
  6. Write your code and tests.
  7. Test it.
  8. Commit your code with -s for signed-off
  9. Submit a pull request to the original repo on your group branch.
  10. Get review from your group leader and partners.
  11. If you are not getting a LGTM, goto 6 and rewrite your code.
  12. Get a LGTM and merge your code.

If you are a group leader, in addition to the above, you need to:

  1. Submit a pull request from your group branch to master.
  2. Get review from @AsterNighT .
  3. If you are not getting a LGTM, go to the code writer and ask him/her to fix it, goto 2.
  4. Get a LGTM and merge your code.

How to do code review?

Go through the code line by line carefully, start a review and ask yourself questions:

  1. Is the code clear enough? If not, does it get enough comments?
  2. Is the code badly composed? It there an obvious way to make it better?
  3. Is the code necessary?
  4. It the code error-prone? Is it fully tested, manually or automatically?
  5. Is there any unit test if necessary?

Don't hesitate to raised questions. It is the author's duty to answer them. Even the mildest suggestion matters. If you don't raise your question now, you may be questioned by @AsterNighT when you try to merge your code into master.

If you feel good enough about the code, apply an approve and comment "LGTM" (Look good to me). This is common practice. If not, feel free to comment and request change from the author.

If the author has fixed the code, he/she can @the reviewer with the word "PTAL" (Please take another look, common practice, too) and ask for a new review.

Precautions and questions you may ask

About

Backend for 2021 ZJU software engineering

Resources

Stars

Watchers

Forks

Releases

No releases published

Packages

No packages published

Languages