-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update README #11
Update README #11
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor typos, otherwise LGTM 🤩
Co-authored-by: Zhong Qing <[email protected]>
Co-authored-by: Zhong Qing <[email protected]>
fixes #10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM less the minor changes! we are required to acknowledge the original source of the code (https://se-education.org). i'm not sure if the last line in README.md is sufficient as acknowledgement, or if it'll even be there after we change it to fit our project.
@@ -1,14 +1,35 @@ | |||
[](https://github.com/se-edu/addressbook-level3/actions) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to change this by generating our own workflow badge. not sure if it's only for Java CI only or for all workflow though..
To begin your journey with NetConnect, simply download the latest release from [here]() and refer to the [User Guide](docs/UserGuide.md) | ||
for detailed guidance in using the application. | ||
|
||
For the detailed documentation of this project, see the **[Address Book Product Website](https://se-education.org/addressbook-level3)**. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think we need to change the description and link of the documentation to ours
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
our group needs to decide whether to use the term "supplier" or "partner"
No description provided.