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.

Sign up for GitHub

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

Jump to bottom

Update DG for Edit Command #67

Merged
merged 3 commits into from
Oct 26, 2023
Merged

Update DG for Edit Command #67

merged 3 commits into from
Oct 26, 2023

Conversation

tanshiyu1999
Copy link

No description provided.

@tanshiyu1999 tanshiyu1999 added type.Documentation Items related to or requiring documentation (custom label) priority.Low Unlikely to do (CS2103T label) severity.Low A flaw that is unlikely to impact the app usage and/or only rarely occurs (CS2103T label) labels Oct 25, 2023
@tanshiyu1999 tanshiyu1999 added this to the v1.3 milestone Oct 25, 2023
@tanshiyu1999 tanshiyu1999 self-assigned this Oct 25, 2023
@codecov
Copy link

codecov bot commented Oct 25, 2023 •
edited
Loading

Codecov Report

Merging #67 (82d7a30) into master (0e7b689) will not change coverage.
The diff coverage is n/a.

❗ Current head 82d7a30 differs from pull request most recent head 7c3e21d. Consider uploading reports for the commit 7c3e21d to get more accurate results

@@            Coverage Diff            @@
##             master      #67   +/-   ##
=========================================
  Coverage     77.67%   77.67%           
  Complexity      636      636           
=========================================
  Files            81       81           
  Lines          1805     1805           
  Branches        235      235           
=========================================
  Hits           1402     1402           
  Misses          321      321           
  Partials         82       82

CJ-Lee01
CJ-Lee01 reviewed Oct 25, 2023
View reviewed changes
docs/DeveloperGuide.md Outdated Show resolved Hide resolved
CJ-Lee01
CJ-Lee01 approved these changes Oct 26, 2023
View reviewed changes
Copy link

@CJ-Lee01 CJ-Lee01 left a comment

Choose a reason for hiding this comment

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

Looks good.

@tanshiyu1999 tanshiyu1999 merged commit 34686ea into AY2324S1-CS2103T-W08-3:master Oct 26, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CJ-Lee01 CJ-Lee01 CJ-Lee01 approved these changes

@McNaBry McNaBry Awaiting requested review from McNaBry

@wxwern wxwern Awaiting requested review from wxwern

@wamps-jp wamps-jp Awaiting requested review from wamps-jp

Assignees

@tanshiyu1999 tanshiyu1999

Labels
priority.Low Unlikely to do (CS2103T label) severity.Low A flaw that is unlikely to impact the app usage and/or only rarely occurs (CS2103T label) type.Documentation Items related to or requiring documentation (custom label)
Projects
None yet
Milestone
v1.3
Development

Successfully merging this pull request may close these issues.
2 participants
@tanshiyu1999 @CJ-Lee01