Skip to content

✨ [maykinmedia/open-api-framework#163] Integrate django-common#665

Merged
stevenbal merged 9 commits intomasterfrom
feature/add-maykin-common
Oct 16, 2025
Merged

✨ [maykinmedia/open-api-framework#163] Integrate django-common#665
stevenbal merged 9 commits intomasterfrom
feature/add-maykin-common

Conversation

@danielmursa-dev
Copy link
Contributor

@danielmursa-dev danielmursa-dev commented Sep 12, 2025

@codecov-commenter
Copy link

codecov-commenter commented Sep 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.98%. Comparing base (66f7dd2) to head (36a0e92).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #665      +/-   ##
==========================================
+ Coverage   82.08%   82.98%   +0.90%     
==========================================
  Files         131      128       -3     
  Lines        2489     2422      -67     
  Branches      201      192       -9     
==========================================
- Hits         2043     2010      -33     
+ Misses        403      371      -32     
+ Partials       43       41       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@danielmursa-dev danielmursa-dev force-pushed the feature/add-maykin-common branch 2 times, most recently from 160a047 to 1cf585f Compare October 7, 2025 13:35
@danielmursa-dev danielmursa-dev requested review from Floris272, TimdeBeer1 and stevenbal and removed request for Floris272 and stevenbal October 7, 2025 13:39
@danielmursa-dev danielmursa-dev force-pushed the feature/add-maykin-common branch from 8990f21 to b86fbe9 Compare October 8, 2025 14:12
Copy link
Collaborator

@stevenbal stevenbal left a comment

Choose a reason for hiding this comment

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

Same issue with styling in docker seems to occur EDIT seemed to be a local caching issue with the image
image

@danielmursa-dev danielmursa-dev force-pushed the feature/add-maykin-common branch from 362e85d to 36a0e92 Compare October 14, 2025 09:07
@github-actions
Copy link
Contributor

github-actions bot commented Oct 14, 2025

🐰 Bencher Report

Branchfeature/add-maykin-common
Testbedubuntu-latest
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
milliseconds (ms)
(Result Δ%)
Upper Boundary
milliseconds (ms)
(Limit %)
performance_test/tests/test_objects_list.py::test_objects_api_list_filter_one_result📈 view plot
🚷 view threshold
19.26 ms
(-3.27%)Baseline: 19.91 ms
20.91 ms
(92.13%)
performance_test/tests/test_objects_list.py::test_objects_api_list_large_page_size_page_1📈 view plot
🚷 view threshold
231.37 ms
(+0.71%)Baseline: 229.73 ms
241.22 ms
(95.92%)
performance_test/tests/test_objects_list.py::test_objects_api_list_large_page_size_page_5📈 view plot
🚷 view threshold
234.34 ms
(-0.08%)Baseline: 234.52 ms
246.25 ms
(95.16%)
performance_test/tests/test_objects_list.py::test_objects_api_list_small_page_size_page_20📈 view plot
🚷 view threshold
95.16 ms
(-0.62%)Baseline: 95.76 ms
100.55 ms
(94.64%)
🐰 View full continuous benchmarking report in Bencher

@stevenbal stevenbal merged commit 6b0b290 into master Oct 16, 2025
27 checks passed
@stevenbal stevenbal deleted the feature/add-maykin-common branch October 16, 2025 09:40
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.

Integrate django-common package into all components

3 participants