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?

iulianpascalau and others added 30 commits June 8, 2023 11:00
Manage multiple network messengers on network components
small refactor on antiflood components creation
created a new full history peer shard mapper
create new interceptors for heartbeat to feed the new peer shard mapper
Full archive heartbeat sender + new data pools
added separate preferred peers holder for the new network
- added new p2p lib version
…//github.com/multiversx/mx-chain-go into requesters_resolvers_full_archive_network

# Conflicts:
#	factory/processing/processComponents.go
#	factory/processing/processComponentsHandler_test.go
#	integrationTests/multiShard/hardFork/hardFork_test.go
#	update/factory/exportHandlerFactory.go
sstanculeanu and others added 23 commits July 20, 2023 15:48
…le_p2p_messengers

Merge rc160 into feat multiple p2p messengers
Update mx-chain-communication-go tag
# Conflicts:
#	go.mod
#	go.sum
#	outport/outport.go
# Conflicts:
#	cmd/node/config/enableEpochs.toml
#	cmd/seednode/main.go
#	common/enablers/enableEpochsHandler.go
#	common/enablers/enableEpochsHandler_test.go
#	common/enablers/epochFlags.go
#	common/interface.go
#	config/epochConfig.go
#	config/tomlConfig_test.go
#	factory/network/networkComponents.go
#	factory/network/networkComponents_test.go
#	go.mod
#	go.sum
#	integrationTests/testInitializer.go
#	p2p/constants.go
#	sharding/mock/enableEpochsHandlerMock.go
#	testscommon/enableEpochsHandlerMock/enableEpochsHandlerStub.go
# Conflicts:
#	common/interface.go
#	factory/core/coreComponents.go
#	factory/core/coreComponentsHandler.go
#	factory/mock/coreComponentsMock.go
#	node/mock/factory/coreComponentsStub.go
#	process/errors.go
#	process/mock/coreComponentsMock.go
#	testscommon/factory/coreComponentsHolderStub.go
@codecov
Copy link

codecov bot commented Aug 11, 2023

Codecov Report

Patch coverage: 91.66% and project coverage change: +9.15% 🎉

Comparison is base (80d86a8) 70.91% compared to head (68d0de4) 80.06%.
Report is 1302 commits behind head on feat/consensus-size-changes.

❗ Current head 68d0de4 differs from pull request most recent head 08a877c. Consider uploading reports for the commit 08a877c to get more accurate results

Additional details and impacted files
@@                       Coverage Diff                       @@
##           feat/consensus-size-changes    #5475      +/-   ##
===============================================================
+ Coverage                        70.91%   80.06%   +9.15%     
===============================================================
  Files                              677      708      +31     
  Lines                            87950    93651    +5701     
===============================================================
+ Hits                             62370    74984   +12614     
+ Misses                           20897    13331    -7566     
- Partials                          4683     5336     +653     
Files Changed Coverage Δ
api/groups/validatorGroup.go 100.00% <ø> (+24.48%) ⬆️
api/middleware/responseLogger.go 66.66% <0.00%> (ø)
cmd/assessment/benchmarks/erc20Benchmark.go 0.00% <ø> (ø)
cmd/assessment/benchmarks/wasmBenchmark.go 0.00% <ø> (ø)
cmd/keygenerator/converter/pidPubkeyConverter.go 90.47% <ø> (ø)
...termui/view/termuic/termuiRenders/widgetsRender.go 2.94% <0.00%> (-0.04%) ⬇️
common/channels.go 57.14% <0.00%> (-42.86%) ⬇️
common/trie.go 46.15% <0.00%> (-37.18%) ⬇️
...ataRetriever/dataPool/headersCache/headersCache.go 90.15% <ø> (+10.60%) ⬆️
dataRetriever/resolvers/headerResolver.go 100.00% <ø> (+9.52%) ⬆️
... and 69 more

... and 236 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 32c3f05 into feat/consensus-size-changes Aug 11, 2023
@bogdan-rosianu bogdan-rosianu deleted the merge-rc-v1-6-cons-changes-aug11 branch August 11, 2023 10:37
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.

10 participants