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

[FlowContractAudits] Add function to get FlowContractAudits contract #275

Merged
merged 5 commits into from
Feb 2, 2022

Conversation

nvdtf
Copy link
Member

@nvdtf nvdtf commented Feb 2, 2022

  • Need getter function for FlowContractAudits.cdcon FVM bootstrap for deployment. This was missed since the test suites were different.
  • Fixed CI. There is new flow-go/crypto release v0.24.2 (today) that fails the CI run. The build script has hardcoded the version but had a minor issue in tagging the corresponding Go module. Also tried fixing tests with v0.24.2 but is not compatible with the version of flow-go in the repo. Probably will need to address crypto package upgrade in a later PR.

@nvdtf nvdtf requested a review from joshuahannan as a code owner February 2, 2022 01:24
@joshuahannan joshuahannan merged commit 842c892 into master Feb 2, 2022
@joshuahannan joshuahannan deleted the navid/contract-audits-contract-function branch February 2, 2022 15:52
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.

None yet

2 participants