Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: remove unnecessary validator config settings #4814

Conversation

jorgeantonio21
Copy link
Contributor

Description

Tackle tari-dan issue.

Motivation and Context

How Has This Been Tested?

@jorgeantonio21 jorgeantonio21 marked this pull request as ready for review October 19, 2022 16:41
@CjS77 CjS77 added the P-acks_required Process - Requires more ACKs or utACKs label Oct 20, 2022
@stringhandler stringhandler added the P-merge Process - Queued for merging label Oct 20, 2022
@stringhandler
Copy link
Collaborator

utACK

@CjS77 CjS77 removed the P-acks_required Process - Requires more ACKs or utACKs label Oct 20, 2022
Copy link
Collaborator

@SWvheerden SWvheerden left a comment

Choose a reason for hiding this comment

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

ACK

@sdbondi
Copy link
Member

sdbondi commented Oct 21, 2022

ACK

@CjS77 CjS77 merged commit b303ec5 into tari-project:development Oct 21, 2022
stringhandler pushed a commit that referenced this pull request Oct 25, 2022
Description
---
Removes unused config from validator node

Motivation and Context
---
Fixes #4814 

How Has This Been Tested?
---
Validator node starts up
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P-merge Process - Queued for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants