Skip to content

Conversation

@miiu96
Copy link
Contributor

@miiu96 miiu96 commented Mar 3, 2025

Reasoning behind the pull request

  • Update chain simulator implementation for Andromeda

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?

oneEGLD = big.NewInt(1000000000000000000)
)

func TestMainRunRUN(t *testing.T) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this needed?

Copy link
Contributor Author

@miiu96 miiu96 Mar 3, 2025

Choose a reason for hiding this comment

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

no, reverted the changes

) func(t *testing.T) {
return func(t *testing.T) {
_ = removeANSIColorsForLoggerIfNeeded()
_ = logger.SetLogLevel("*:DEBUG")
Copy link
Collaborator

Choose a reason for hiding this comment

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

are these needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no, removed

Comment on lines 87 to 88
notNil := !check.IfNil(currentHeader)
enableEpochHandler := creator.nodeHandler.GetCoreComponents().EnableEpochsHandler()
Copy link
Collaborator

Choose a reason for hiding this comment

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

perhaps move them where they are used

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@miiu96 miiu96 merged commit a51780b into feat/andromeda-fixes Mar 5, 2025
4 checks passed
@miiu96 miiu96 deleted the chain-simulator-integration-andromeda branch March 5, 2025 14:12
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.

4 participants