Skip to content

WT-17203 ccache not being used in build#13561

Merged
ablekhman merged 1 commit intodevelopfrom
wt-17203-ccache
Apr 17, 2026
Merged

WT-17203 ccache not being used in build#13561
ablekhman merged 1 commit intodevelopfrom
wt-17203-ccache

Conversation

@ablekhman
Copy link
Copy Markdown
Contributor

  • Fix typo in cmake scripts

- Fix typo in cmake scripts
@ablekhman ablekhman requested a review from a team as a code owner April 17, 2026 06:07
@ablekhman ablekhman requested a review from mana2-bot April 17, 2026 06:07
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 17, 2026

Thanks for creating a pull request! Please answer the questions below by editing this comment — completing this upfront typically speeds up the review process.

Checklist before requesting a review

  • I have performed a self-review of my code.
  • I can point to specific tests that would catch regressions of this change, or I've documented why existing test coverage is sufficient.
  • I have made corresponding changes to the documentation, or no documentation change is needed.

What makes this change safe?

Help reviewers focus their attention by answering these questions:

  • Data format compatibility: Does this change modify the format of WT data or metadata? If so, does this change support non-disruptive upgrade of older systems?
  • Test coverage: Which specific tests validate your change? (Name the test files/functions that would fail if your change were reverted)
  • Risk assessment: What functionality could break if this change has bugs? How thoroughly have you tested edge cases?
  • Review focus: What aspects would you want a teammate to double-check?

Type of change made in this PR

  • Functional change
  • Test-only change
  • Refactor-only change
  • Other non-functional change

References:

@ablekhman ablekhman requested a review from lukech April 17, 2026 06:09
Copy link
Copy Markdown
Contributor

@lukech lukech left a comment

Choose a reason for hiding this comment

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

LGTM

@ablekhman ablekhman enabled auto-merge April 17, 2026 06:11
Copy link
Copy Markdown
Contributor

@etienneptl etienneptl left a comment

Choose a reason for hiding this comment

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

Nice!

@ablekhman ablekhman added this pull request to the merge queue Apr 17, 2026
Merged via the queue into develop with commit af950ff Apr 17, 2026
16 checks passed
@ablekhman ablekhman deleted the wt-17203-ccache branch April 17, 2026 08:32
LX6T pushed a commit that referenced this pull request Apr 20, 2026
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.

3 participants