Skip to content

Conversation

kruplm
Copy link
Contributor

@kruplm kruplm commented Mar 14, 2023

  • Simplified ProcessArgs() by processing entries dynamically.
  • Renamed files to follow camelCase across the project

psmulovics
psmulovics previously approved these changes Mar 14, 2023
@codecov
Copy link

codecov bot commented Mar 14, 2023

Codecov Report

Merging #178 (60a8325) into main (4421d52) will increase coverage by 0.10%.
The diff coverage is 66.66%.

❗ Current head 60a8325 differs from pull request most recent head 16754c1. Consider uploading reports for the commit 16754c1 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #178      +/-   ##
==========================================
+ Coverage   79.89%   79.99%   +0.10%     
==========================================
  Files          82       82              
  Lines        3198     3184      -14     
  Branches      139      135       -4     
==========================================
- Hits         2555     2547       -8     
+ Misses        643      637       -6     
Impacted Files Coverage Δ
...c/shell/js/composeui-node-launcher/src/launcher.ts 62.50% <66.66%> (ø)

... and 1 file with indirect coverage changes

@kruplm kruplm marked this pull request as ready for review March 14, 2023 13:05
@kruplm kruplm requested a review from a team March 14, 2023 13:05
@kruplm kruplm merged commit f438155 into morganstanley:main Mar 14, 2023
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