Skip to content

feat(scripts): check version of the compilation tools before build(#1295)#1307

Merged
mergify[bot] merged 1 commit intostoneatom:stonedb-5.7-devfrom
davidshiz:mtr
Feb 15, 2023
Merged

feat(scripts): check version of the compilation tools before build(#1295)#1307
mergify[bot] merged 1 commit intostoneatom:stonedb-5.7-devfrom
davidshiz:mtr

Conversation

@davidshiz
Copy link
Copy Markdown
Collaborator

@davidshiz davidshiz commented Feb 15, 2023

Summary about this PR

Issue Number: close #1295

Tests Check List

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Changelog

  • New Feature
  • Bug Fix
  • Performance Improvement
  • Build/Testing/CI/CD
  • Documentation
  • Not for changelog (changelog entry is not required)

Documentation

  • Affects user behaviors
  • Contains syntax changes
  • Contains variable changes
  • Contains experimental features

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Feb 15, 2023

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

@mergify mergify bot added the PR-feature feature for pull request label Feb 15, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 15, 2023

Codecov Report

Base: 43.13% // Head: 43.13% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (117897e) compared to base (36fbc22).
Patch has no changes to coverable lines.

Additional details and impacted files
@@               Coverage Diff                @@
##           stonedb-5.7-dev    #1307   +/-   ##
================================================
  Coverage            43.13%   43.13%           
================================================
  Files                 1830     1830           
  Lines               396271   396271           
================================================
+ Hits                170925   170931    +6     
+ Misses              225346   225340    -6     
Impacted Files Coverage Δ
storage/innobase/include/ut0mutex.ic 52.38% <0.00%> (-19.05%) ⬇️
storage/innobase/row/row0vers.cc 71.66% <0.00%> (-4.00%) ⬇️
storage/innobase/trx/trx0rec.cc 80.00% <0.00%> (-2.70%) ⬇️
storage/innobase/sync/sync0rw.cc 98.88% <0.00%> (-1.12%) ⬇️
storage/tianmu/compress/top_bit_dict.cpp 66.15% <0.00%> (-0.66%) ⬇️
storage/innobase/row/row0upd.cc 77.96% <0.00%> (-0.64%) ⬇️
storage/innobase/buf/buf0flu.cc 85.14% <0.00%> (-0.64%) ⬇️
sql/sql_profile.cc 10.30% <0.00%> (-0.43%) ⬇️
sql/sql_error.cc 77.39% <0.00%> (-0.35%) ⬇️
storage/innobase/row/row0ftsort.cc 69.49% <0.00%> (-0.34%) ⬇️
... and 20 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mergify mergify bot merged commit fa5917a into stoneatom:stonedb-5.7-dev Feb 15, 2023
@davidshiz davidshiz deleted the mtr branch February 15, 2023 13:11
@Nliver Nliver added this to the StoneDB_5.7_v1.0.3 milestone Apr 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR-feature feature for pull request

Projects

Development

Successfully merging this pull request may close these issues.

feature: Optimize stonedb build script to check compilation tool or other dependencies version

4 participants