-
Notifications
You must be signed in to change notification settings - Fork 44
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
front: nge language synchro + Remove SBB header + bump NGE version 2.9.9 #9760
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## dev #9760 +/- ##
==========================================
- Coverage 37.82% 37.80% -0.03%
==========================================
Files 994 994
Lines 91105 91147 +42
Branches 1176 1176
==========================================
- Hits 34463 34455 -8
- Misses 56188 56238 +50
Partials 454 454
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
c82b079
to
8b08a08
Compare
8b08a08
to
81ed0a8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, tested just a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm and tested, good job !
Maybe we can wait for @emersion or @flomonster approval for the license part to merge :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM in any case
ac636b1
to
67d7793
Compare
Signed-off-by: Louis Greiner <greiner.louis@gmail.com>
67d7793
to
b61f6a6
Compare
Close #9694
note to myself : remember to squash before merge