Skip to content

Conversation

@echevrier
Copy link
Contributor

used by the exchange rate oracle. integritee-network/worker#292

Copy link
Collaborator

@haerdib haerdib left a comment

Choose a reason for hiding this comment

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

Does this pallet not need any benchmark weights?

"pallet-treasury/runtime-benchmarks",
"pallet-vesting/runtime-benchmarks",
"pallet-utility/runtime-benchmarks",
#"pallet-teeracle/runtime-benchmarks",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this pallet not be benchmarked?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. I will comment it out in the next task. #65

[package.metadata.docs.rs]
targets = ["x86_64-unknown-linux-gnu"]

# local dependencies
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 comment still valid?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not really. I will remove it

Copy link
Collaborator

@haerdib haerdib left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@haerdib haerdib merged commit 6965648 into master Nov 4, 2021
@brenzi
Copy link
Collaborator

brenzi commented Nov 4, 2021

love the freestyle wordplay in the naming 👍

@echevrier echevrier deleted the exchange_rate_oracle branch April 4, 2022 13:01
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