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 Tag References #267

Merged
merged 1 commit into from
Nov 11, 2023
Merged

Conversation

RB9823
Copy link

@RB9823 RB9823 commented Nov 11, 2023

No description provided.

Copy link

codecov bot commented Nov 11, 2023

Codecov Report

Merging #267 (9d5a013) into master (95a8b8c) will decrease coverage by 0.15%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #267      +/-   ##
============================================
- Coverage     70.02%   69.87%   -0.15%     
+ Complexity      567      564       -3     
============================================
  Files            94       94              
  Lines          1888     1882       -6     
  Branches        215      214       -1     
============================================
- Hits           1322     1315       -7     
- Misses          489      490       +1     
  Partials         77       77              
Files Coverage Δ
...in/java/seedu/address/logic/parser/ParserUtil.java 93.18% <ø> (-0.82%) ⬇️

... and 1 file with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@markgcera
Copy link

LGTM. Code coverage has dropped but this is a necessary change for code quality

@markgcera markgcera merged commit 33481bf into AY2324S1-CS2103T-F10-1:master Nov 11, 2023
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants