Skip to content

Comments

chore: error type for block header#731

Merged
rakita merged 3 commits intobluealloy:mainfrom
hack3r-0m:main
Sep 21, 2023
Merged

chore: error type for block header#731
rakita merged 3 commits intobluealloy:mainfrom
hack3r-0m:main

Conversation

@hack3r-0m
Copy link
Contributor

should close #685

I have used enum name as header since env names such as BlockEnv, CfgEnv & Env are already used as type, let me know if prefer EnvError over this and I can quickly make that change.

}
}

#[derive(Debug, Copy, Clone, PartialEq, Eq)]
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
#[derive(Debug, Copy, Clone, PartialEq, Eq)]
/// Errors related to misconfiguration of the `BlockEnv`
#[derive(Debug, Copy, Clone, PartialEq, Eq)]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

resolved in 498d24c

Copy link
Member

@rakita rakita left a comment

Choose a reason for hiding this comment

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

lgtm, one nit

@rakita
Copy link
Member

rakita commented Sep 20, 2023

@hack3r-0m can you resolve the conflict, so that we can merge it?

@gakonst
Copy link
Collaborator

gakonst commented Sep 20, 2023

(failing CI fixed by #735)

@hack3r-0m
Copy link
Contributor Author

hack3r-0m commented Sep 21, 2023

rebased it over main

@rakita
Copy link
Member

rakita commented Sep 21, 2023

@hack3r-0m missing include.

@rakita rakita merged commit f72eaa0 into bluealloy:main Sep 21, 2023
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.

Introduce error type for EnvError

3 participants