Skip to content

feat(governance): Log instructions of queries in Governance.#9130

Open
daniel-wong-dfinity-org wants to merge 2 commits intomasterfrom
log-governance-query-instructions-daniel-wong
Open

feat(governance): Log instructions of queries in Governance.#9130
daniel-wong-dfinity-org wants to merge 2 commits intomasterfrom
log-governance-query-instructions-daniel-wong

Conversation

@daniel-wong-dfinity-org
Copy link
Contributor

@daniel-wong-dfinity-org daniel-wong-dfinity-org commented Mar 2, 2026

This creates a #[query] attribute that wraps the one in ic_cdk. All this new #[query] attribute does is add some logging at the end so that we can track performance.

This PR was mostly written by AI.

@daniel-wong-dfinity-org daniel-wong-dfinity-org marked this pull request as ready for review March 2, 2026 21:46
@daniel-wong-dfinity-org daniel-wong-dfinity-org requested a review from a team as a code owner March 2, 2026 21:46
@github-actions github-actions bot added the feat label Mar 2, 2026
github-actions[bot]

This comment was marked as resolved.

@daniel-wong-dfinity-org daniel-wong-dfinity-org dismissed github-actions[bot]’s stale review March 2, 2026 21:48
  1. This just adds some printlns, so I'd say that there are no "behavior" changes.
  2. No.
  3. Existing data is fine.
  4. Seems like very low risk.
auto-merge was automatically disabled March 2, 2026 22:50

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants