Skip to content
This repository has been archived by the owner on Jan 10, 2025. It is now read-only.

Governance: voter-weight-addin cleanup #2512

Merged
merged 9 commits into from
Oct 19, 2021
Merged

Governance: voter-weight-addin cleanup #2512

merged 9 commits into from
Oct 19, 2021

Conversation

SebastianBor
Copy link
Contributor

@SebastianBor SebastianBor commented Oct 14, 2021

Summary

This PR address feedback from #2450

  1. If spl_governance_voter_weight_addin.so doesn't exist it's built before running governance tests.
  2. spl_governance_voter_weight_addin doesn't have dependency on spl_governnace_chatany longer. spl_governance_tools crate was create for shared utilities in the governance programs ecosystem.
  3. reserved fields on RealmConfigAccount renamed to concrete add-ins they are reserved for

fix: build addin during test run

fix: build voter weight addin for tests using the addin only

chore: use mutex to build addin only once

chore: move build guard to separate file

chore: update governance version for chat
@SebastianBor SebastianBor changed the title Governance: use voter-weight-addin as dev-dependency Governance: voter-weight-addin cleanup Oct 16, 2021
@SebastianBor SebastianBor marked this pull request as ready for review October 16, 2021 13:27
Copy link
Contributor

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

Just some nits, looks good otherwise

governance/program/tests/program_test/addins.rs Outdated Show resolved Hide resolved
governance/program/src/state/realm_config.rs Outdated Show resolved Hide resolved
Copy link
Contributor

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

Just one more question regarding backwards compatibility, otherwise this is good to go

governance/program/src/error.rs Show resolved Hide resolved
@SebastianBor SebastianBor merged commit d6d0b92 into solana-labs:master Oct 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants