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

HTM-1225: implement prioritising job triggers #984

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

mprins
Copy link
Contributor

@mprins mprins commented Oct 10, 2024

HTM-1225 Powered by Pull Request Badge

Quartz default value is 5, we should stick with positive values

@mprins mprins added the enhancement New feature or request label Oct 10, 2024
@mprins mprins self-assigned this Oct 10, 2024
Copy link

codecov bot commented Oct 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Impacted file tree graph

@@          Coverage Diff          @@
##             main   #984   +/-   ##
=====================================
+ Coverage      77%    78%   +1%     
- Complexity     82     86    +4     
=====================================
  Files         107    107           
  Lines        4818   4829   +11     
  Branches      440    440           
=====================================
+ Hits         3708   3719   +11     
  Misses        877    877           
  Partials      233    233           
Files with missing lines Coverage Δ Complexity Δ
...lormap/api/configuration/dev/PopulateTestData.java 91% <100%> (+1%) 0 <0> (ø)
...ava/org/tailormap/api/scheduling/TMJobDataMap.java 83% <100%> (+8%) 8 <4> (+4)
...java/org/tailormap/api/scheduling/TaskCreator.java 82% <100%> (+1%) 0 <0> (ø)

Copy link

github-actions bot commented Oct 10, 2024

Test Results

277 tests  ±0   276 ✅ ±0   3m 23s ⏱️ +31s
 29 suites ±0     1 💤 ±0 
 29 files   ±0     0 ❌ ±0 

Results for commit 67a48de. ± Comparison against base commit da071b2.

♻️ This comment has been updated with latest results.

@mprins mprins force-pushed the HTM-1225_implement_prioritising branch from c9e04e0 to 3e1cda3 Compare October 11, 2024 09:39
@mprins mprins marked this pull request as ready for review October 11, 2024 09:41
Quartz default value is 5, we should stick with positive values
@mprins mprins force-pushed the HTM-1225_implement_prioritising branch from 3e1cda3 to 67a48de Compare October 11, 2024 09:41
@mprins mprins enabled auto-merge October 11, 2024 09:45
@mprins mprins disabled auto-merge October 11, 2024 12:08
@mprins mprins merged commit 497b032 into main Oct 11, 2024
26 checks passed
@mprins mprins deleted the HTM-1225_implement_prioritising branch October 11, 2024 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant