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

fix(#238): added RepeatDirection to Panels #239

Merged
merged 4 commits into from
Dec 19, 2023

Conversation

fasibio
Copy link

@fasibio fasibio commented Dec 15, 2023

Is depent on K-Phoen/sdk#26. And also versionupdate from sdk is needed first.

Will make it possible to set RepeatDirection.

Copy link

codecov bot commented Dec 15, 2023

Codecov Report

Merging #239 (9566b07) into master (6867a4a) will decrease coverage by 0.39%.
Report is 1 commits behind head on master.
The diff coverage is 60.43%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #239      +/-   ##
==========================================
- Coverage   90.08%   89.70%   -0.39%     
==========================================
  Files          92       92              
  Lines        6941     7032      +91     
==========================================
+ Hits         6253     6308      +55     
- Misses        543      572      +29     
- Partials      145      152       +7     
Files Coverage Δ
gauge/gauge.go 100.00% <100.00%> (ø)
graph/graph.go 98.41% <100.00%> (+0.03%) ⬆️
heatmap/heatmap.go 100.00% <100.00%> (ø)
logs/logs.go 100.00% <100.00%> (ø)
singlestat/singlestat.go 100.00% <100.00%> (ø)
stat/stat.go 100.00% <100.00%> (ø)
timeseries/timeseries.go 99.12% <100.00%> (+0.02%) ⬆️
decoder/heatmap.go 80.00% <50.00%> (-1.66%) ⬇️
decoder/logs.go 93.54% <50.00%> (-3.01%) ⬇️
decoder/timeseries.go 73.15% <50.00%> (-0.42%) ⬇️
... and 5 more

@fasibio
Copy link
Author

fasibio commented Dec 15, 2023

Current changes at vendor was made from hand. Is obsolate after sdk pr K-Phoen/sdk#26 is merged

@K-Phoen K-Phoen merged commit da61158 into K-Phoen:master Dec 19, 2023
3 of 5 checks passed
@K-Phoen
Copy link
Owner

K-Phoen commented Dec 19, 2023

Merged, thanks :)

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