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

update ug #206

Merged
merged 3 commits into from
Nov 10, 2023
Merged

update ug #206

merged 3 commits into from
Nov 10, 2023

Conversation

nicolengk
Copy link

No description provided.

@nicolengk nicolengk added type.Task Something that needs to be done, but not a story, bug, or an epic. e.g. Move testing code into a new priority.High Must do labels Nov 10, 2023
@nicolengk nicolengk added this to the v1.4 milestone Nov 10, 2023
@nicolengk nicolengk self-assigned this Nov 10, 2023
Copy link

codecov bot commented Nov 10, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

Copy link

@jianrong7 jianrong7 left a comment

Choose a reason for hiding this comment

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

Some changes 👍

Comment on lines 680 to 683
**Q**: How do I open Terminal?<br>
**A**: If you are a Windows user, press the Windows key + R to open the Run window, then type wt and click OK.<br>
If you are using a Mac, click the Launchpad icon in the Dock, type Terminal in the search field, then click Terminal.

Choose a reason for hiding this comment

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

Add for Linux

Comment on lines 331 to 332
### Delete Commands
[Back to Table of Contents](#table-of-contents)

Choose a reason for hiding this comment

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

You added an empty line in between these 2 lines for the commands above.

Comment on lines 370 to 371
### Edit Commands
[Back to Table of Contents](#table-of-contents)

Choose a reason for hiding this comment

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

You added an empty line in between these 2 lines for the commands above.

Comment on lines 435 to 436
### Find Commands
[Back to Table of Contents](#table-of-contents)

Choose a reason for hiding this comment

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

You added an empty line in between these 2 lines for the commands above.

Comment on lines 480 to 481
### Filter Commands
[Back to Table of Contents](#table-of-contents)

Choose a reason for hiding this comment

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

You added an empty line in between these 2 lines for the commands above.

Copy link

@jianrong7 jianrong7 left a comment

Choose a reason for hiding this comment

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

OK

@nicolengk nicolengk merged commit e66c701 into AY2324S1-CS2103T-W11-2:master Nov 10, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority.High Must do type.Task Something that needs to be done, but not a story, bug, or an epic. e.g. Move testing code into a new
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants