Skip to content

Conversation

@bogdan-rosianu
Copy link
Contributor

Reasoning behind the pull request

  • merge

Proposed changes

  • merge

Testing procedure

  • n/a

Pre-requisites

Based on the Contributing Guidelines the PR author and the reviewers must check the following requirements are met:

  • was the PR targeted to the correct branch?
  • if this is a larger feature that probably needs more than one PR, is there a feat branch created?
  • if this is a feat branch merging, do all satellite projects have a proper tag inside go.mod?

AdoAdoAdo and others added 30 commits January 20, 2023 14:32
…order-integration

# Conflicts:
#	common/ordering/orederedCollection.go
integrate execution order with outport driver
# Conflicts:
#	common/interface.go
#	errors/errors.go
#	factory/processing/processComponents.go
#	integrationTests/testProcessorNode.go
#	node/nodeRunner.go
#	testscommon/components/components.go
iulianpascalau and others added 24 commits August 30, 2023 16:16
Enable sharded storage for user accounts and peer accounts
set max gas per query to 300s equivalent
…-search

Extended the search for the full history resolver
# Conflicts:
#	process/smartContract/scQueryService.go
#	process/smartContract/scQueryService_test.go
…ble-check

Enable transfer back no payable check
…-execution

Feat/transaction ordering on execution
Linter fixes for rc/v1.6.0 - 2023.08.30
# Conflicts:
#	common/interface.go
#	config/tomlConfig_test.go
#	process/errors.go
@codecov
Copy link

codecov bot commented Sep 4, 2023

Codecov Report

Patch coverage: 85.30% and project coverage change: -0.01% ⚠️

Comparison is base (f2281f9) 80.06% compared to head (916af56) 80.06%.
Report is 1 commits behind head on feat/consensus-size-changes.

❗ Current head 916af56 differs from pull request most recent head d5b9d22. Consider uploading reports for the commit d5b9d22 to get more accurate results

Additional details and impacted files
@@                       Coverage Diff                       @@
##           feat/consensus-size-changes    #5557      +/-   ##
===============================================================
- Coverage                        80.06%   80.06%   -0.01%     
===============================================================
  Files                              710      708       -2     
  Lines                            93724    93970     +246     
===============================================================
+ Hits                             75043    75237     +194     
- Misses                           13346    13380      +34     
- Partials                          5335     5353      +18     
Files Changed Coverage Δ
api/middleware/responseLogger.go 66.66% <0.00%> (ø)
outport/process/factory/check.go 80.00% <0.00%> (-5.72%) ⬇️
process/block/preprocess/basePreProcess.go 79.57% <0.00%> (-0.35%) ⬇️
process/smartContract/hooks/blockChainHook.go 92.24% <0.00%> (-0.40%) ⬇️
...ocess/transactionEvaluator/transactionSimulator.go 80.76% <0.00%> (ø)
genesis/process/genesisBlockCreator.go 65.60% <50.00%> (-0.17%) ⬇️
process/block/preprocess/transactions.go 60.62% <57.14%> (-0.03%) ⬇️
process/coordinator/process.go 75.82% <71.42%> (-0.03%) ⬇️
factory/api/apiResolverFactory.go 83.08% <71.60%> (-10.76%) ⬇️
process/smartContract/scQueryService.go 79.01% <72.60%> (-8.49%) ⬇️
... and 52 more

... and 2 files with indirect coverage changes

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

@bogdan-rosianu bogdan-rosianu merged commit b74dcf2 into feat/consensus-size-changes Sep 4, 2023
@bogdan-rosianu bogdan-rosianu deleted the merge-rc-1-6-into-feat-cns-size-sep4 branch September 4, 2023 08:52
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.