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 FeedTime validation Regex pattern #286

Merged

Conversation

koonweee
Copy link

@koonweee koonweee commented Nov 1, 2020

Resolve #248
Resolve #260
Resolve #245
Resolve #240

@codecov
Copy link

codecov bot commented Nov 1, 2020

Codecov Report

Merging #286 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #286   +/-   ##
=========================================
  Coverage     74.79%   74.79%           
  Complexity      544      544           
=========================================
  Files            83       83           
  Lines          1579     1579           
  Branches        158      158           
=========================================
  Hits           1181     1181           
  Misses          340      340           
  Partials         58       58           
Impacted Files Coverage Δ Complexity Δ
...in/java/seedu/zookeep/model/feedtime/FeedTime.java 84.61% <ø> (ø) 6.00 <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 ff14f91...892621a. Read the comment docs.

@koonweee
Copy link
Author

koonweee commented Nov 1, 2020

Resolves #240, #245 and #260 as well

@aizatazhar aizatazhar added this to the v1.4 milestone Nov 1, 2020
@aizatazhar aizatazhar merged commit 252be21 into AY2021S1-CS2103T-W15-4:master Nov 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants