Skip to content

fix(eap): Do not autocomplete timestamp #95886

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 21, 2025

Conversation

Zylphrex
Copy link
Member

Timestamps aren't realy useful to autocomplete by querying eap. So skip it.

Timestamps aren't realy useful to autocomplete by querying eap. So skip it.
@Zylphrex Zylphrex requested review from a team as code owners July 18, 2025 16:23
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 18, 2025
@Zylphrex Zylphrex merged commit 2d5f81a into master Jul 21, 2025
65 checks passed
@Zylphrex Zylphrex deleted the txiao/fix/do-not-autocomplete-timestamp branch July 21, 2025 14:48
Copy link

codecov bot commented Jul 21, 2025

Codecov Report

Attention: Patch coverage is 42.85714% with 4 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...dpoints/test_organization_trace_item_attributes.py 20.00% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #95886       +/-   ##
===========================================
+ Coverage   38.39%   81.49%   +43.09%     
===========================================
  Files        9954    10532      +578     
  Lines      557967   608554    +50587     
  Branches    23981    23981               
===========================================
+ Hits       214247   495956   +281709     
+ Misses     343436   112314   -231122     
  Partials      284      284               

andrewshie-sentry pushed a commit that referenced this pull request Jul 21, 2025
Timestamps aren't realy useful to autocomplete by querying eap. So skip
it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants