Skip to content

Conversation

@asmisha
Copy link
Contributor

@asmisha asmisha commented Mar 3, 2025

Check List

  • Tests have been run in packages where changes made if available
  • Linter has been run for changed code
  • Tests for the changes have been added if not covered yet
  • Docs have been added / updated if required

Issue Reference this PR resolves

[For example #12]

Description of Changes Made (if issue reference is not provided)

[Description goes here]

@asmisha asmisha requested a review from a team as a code owner March 3, 2025 13:06
@github-actions github-actions bot added the pr:community Contribution from Cube.js community members. label Mar 3, 2025
@asmisha asmisha added backend:server Issues relating to Cube Core's Server and removed pr:community Contribution from Cube.js community members. labels Mar 3, 2025
serverless?: boolean;
allowNodeRequire?: boolean;
semanticLayerSync?: (context: RequestContext) => Promise<BiToolSyncConfig[]> | BiToolSyncConfig[];
fastReloadEnabled?: boolean;
Copy link
Member

Choose a reason for hiding this comment

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

We don't use any prefixes/suffixes for boolean options. I belive it should fastReload

Copy link
Member

Choose a reason for hiding this comment

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

@ovr
Copy link
Member

ovr commented Mar 3, 2025

@codecov
Copy link

codecov bot commented Mar 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.69%. Comparing base (405a604) to head (5596d9b).
Report is 7 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #9294   +/-   ##
=======================================
  Coverage   83.69%   83.69%           
=======================================
  Files         228      228           
  Lines       82016    82016           
=======================================
  Hits        68640    68640           
  Misses      13376    13376           
Flag Coverage Δ
cubesql 83.69% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@asmisha asmisha requested a review from ovr March 3, 2025 14:24
@asmisha asmisha merged commit 9ea0b9c into master Mar 5, 2025
74 checks passed
@asmisha asmisha deleted the chore/fast-reload-option branch March 5, 2025 05:52
asmisha added a commit that referenced this pull request Mar 25, 2025
* chore(fast-reload): support fastReloadEnabled as a server option

* rename and add to python and rust
asmisha added a commit that referenced this pull request Mar 25, 2025
…) (#9385)

* chore(fast-reload): support fastReloadEnabled as a server option

* rename and add to python and rust
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend:server Issues relating to Cube Core's Server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants