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

Fix Incorrect spacing for Add Appointment command result #71

Merged
merged 10 commits into from
Oct 12, 2021

Conversation

gycgabriel
Copy link

Close #62.

@gycgabriel gycgabriel added this to the v1.2 milestone Oct 12, 2021
@gycgabriel gycgabriel added the bugfix Fixes bug label Oct 12, 2021
@gycgabriel gycgabriel requested a review from joshenx October 12, 2021 03:32
@codecov-commenter
Copy link

Codecov Report

Merging #71 (3d890d3) into master (98ee04f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #71   +/-   ##
=========================================
  Coverage     64.27%   64.27%           
  Complexity      418      418           
=========================================
  Files            83       83           
  Lines          1475     1475           
  Branches        152      152           
=========================================
  Hits            948      948           
  Misses          484      484           
  Partials         43       43           
Impacted Files Coverage Δ
.../address/logic/commands/AddAppointmentCommand.java 0.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 98ee04f...3d890d3. Read the comment docs.

Copy link

@joshenx joshenx left a comment

Choose a reason for hiding this comment

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

lgtm ty

@joshenx joshenx merged commit 31a4a2f into AY2122S1-CS2103-W14-1:master Oct 12, 2021
@@ -18,7 +18,7 @@
public static final String MESSAGE_USAGE =
"appt"

Choose a reason for hiding this comment

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

@gycgabriel think can add a space here from #62 raised by @joshenx

Copy link
Author

@gycgabriel gycgabriel Oct 12, 2021

Choose a reason for hiding this comment

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

Thanks for pointing out, new PR created.

gycgabriel added a commit that referenced this pull request Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Fixes bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect spacing for Appointment error statement
4 participants