Skip to content

Conversation

@Nikita-str
Copy link
Contributor

@Nikita-str Nikita-str commented May 27, 2024

Check List

  • Tests has 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

@Nikita-str Nikita-str requested review from a team as code owners May 27, 2024 18:50
@github-actions github-actions bot added the pr:community Contribution from Cube.js community members. label May 27, 2024
@vercel
Copy link

vercel bot commented May 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

8 Ignored Deployments
Name Status Preview Comments Updated (UTC)
examples-angular-dashboard ⬜️ Ignored (Inspect) Visit Preview May 27, 2024 7:04pm
examples-react-d3 ⬜️ Ignored (Inspect) Visit Preview May 27, 2024 7:04pm
examples-react-dashboard ⬜️ Ignored (Inspect) Visit Preview May 27, 2024 7:04pm
examples-react-data-table ⬜️ Ignored (Inspect) Visit Preview May 27, 2024 7:04pm
examples-react-highcharts ⬜️ Ignored (Inspect) Visit Preview May 27, 2024 7:04pm
examples-react-material-ui ⬜️ Ignored (Inspect) Visit Preview May 27, 2024 7:04pm
examples-react-pivot-table ⬜️ Ignored (Inspect) Visit Preview May 27, 2024 7:04pm
examples-vue-query-builder ⬜️ Ignored (Inspect) Visit Preview May 27, 2024 7:04pm

@KSDaemon KSDaemon force-pushed the bugfix-special-names-underscores branch from d98511c to 4d29bcc Compare June 6, 2025 13:59
@codecov
Copy link

codecov bot commented Jun 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.20%. Comparing base (5dd626a) to head (54023f8).
Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #8303   +/-   ##
=======================================
  Coverage   84.20%   84.20%           
=======================================
  Files         230      230           
  Lines       85318    85315    -3     
=======================================
+ Hits        71841    71842    +1     
+ Misses      13477    13473    -4     
Flag Coverage Δ
cubesql 84.20% <100.00%> (+<0.01%) ⬆️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@KSDaemon KSDaemon left a comment

Choose a reason for hiding this comment

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

👍🏻 LGTM

@KSDaemon KSDaemon force-pushed the bugfix-special-names-underscores branch from 5b7d33e to 54023f8 Compare June 25, 2025 16:55
@KSDaemon KSDaemon merged commit 7bb4bdc into master Jun 26, 2025
120 of 121 checks passed
@KSDaemon KSDaemon deleted the bugfix-special-names-underscores branch June 26, 2025 08:42
Frank-TXS pushed a commit to Helge-TXS/cube that referenced this pull request Aug 5, 2025
…ld` names in aliasing (cube-js#8303)

* dont change __user / __cubejoinfield names in aliasing

* cast name to lower for `__cubeJoinField` case

* reapply fix

---------

Co-authored-by: Konstantin Burkalev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:community Contribution from Cube.js community members.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants