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

WIP: Release 4.2.0 - Changes config implementation #125

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

Conversation

ffflabs
Copy link
Collaborator

@ffflabs ffflabs commented Apr 27, 2021

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Apr 27, 2021

Codecov Report

Merging #125 (6762e51) into master (e196d51) will increase coverage by 21.35%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master     #125       +/-   ##
=============================================
+ Coverage     35.81%   57.16%   +21.35%     
+ Complexity     5435     5385       -50     
=============================================
  Files           144      151        +7     
  Lines         17293    17109      -184     
=============================================
+ Hits           6193     9780     +3587     
+ Misses        11100     7329     -3771     
Impacted Files Coverage Δ
src/util/FlagCache.php 0.00% <0.00%> (-100.00%) ⬇️
src/image/FieldArrow.php 0.00% <0.00%> (-100.00%) ⬇️
src/image/ImgData_PushPins.php 0.00% <0.00%> (-100.00%) ⬇️
src/util/Bezier.php 0.00% <0.00%> (-98.12%) ⬇️
src/image/ImgData_Balls.php 0.00% <0.00%> (-91.84%) ⬇️
src/graph/CCBPGraph.php 0.00% <0.00%> (-78.41%) ⬇️
src/plot/FieldPlot.php 0.00% <0.00%> (-72.73%) ⬇️
src/plot/ScatterPlot.php 50.74% <0.00%> (-37.14%) ⬇️
src/themes/AquaTheme.php 33.33% <0.00%> (-21.67%) ⬇️
src/image/ImgData.php 57.14% <0.00%> (-21.43%) ⬇️
... and 160 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@codeclimate
Copy link

codeclimate bot commented Apr 28, 2021

There are too many results to compare

View more on Code Climate.

@sonarcloud
Copy link

sonarcloud bot commented Apr 28, 2021

@j3j5
Copy link

j3j5 commented Aug 15, 2022

Hey, I'd like to help finishing this up, do you have a list of what's pending or some other way for me to know where to start giving a hand?

list($short, $full) = explode('|', strftime('%b|%B', strtotime("2001-${i}-01")));
$this->iShortMonth[$aLocale][] = ucfirst($short);
$this->iMonthName[$aLocale][] = ucfirst($full);
list($short, $full) = explode('|', strftime('%b|%B', strtotime("2001-${i}-01")));

Choose a reason for hiding this comment

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

strftime is deprecated as of php8.1 https://www.php.net/manual/en/function.strftime.php

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants