Robert Hunt (432ae66d) at 18 Mar 16:38
Robert Hunt (4f86a3f6) at 18 Mar 16:37
Merge branch 'drosse/improve-parser' into 'main'
... and 1 more commit
Improve parser by allowing any bare word ( and underscore ) to be recognised as valid token:
EnumLike everywhere — sources can use EnumLike instead of StringEnumLike for enum fields. Previously words with special chars e.g. waiting_for_something would need to be a stringNA
Improve parser by allowing any bare word ( and underscore ) to be recognised as valid token:
EnumLike everywhere — sources can use EnumLike instead of StringEnumLike for enum fields. Previously words with special chars e.g. waiting_for_something would need to be a stringNA
question: @drosse the MR mentions:
Enables
EnumLikeeverywhere — sources can useEnumLikeinstead ofStringEnumLikefor enum fields. Previously words with special chars e.g.waiting_for_somethingwould need to be a string
But we don't action on it in this MR. I just wanted to confirm if this was missing, or it's more an FYI?
Robert Hunt (b9c10f68) at 18 Mar 15:55
fix: update CodeSuggestions GraphQL structure to match backend impl...
... and 5 more commits
Robert Hunt (175c6535) at 18 Mar 15:44
feat: add orderBy conversion for analytics aggregation queries
FILTER=ai_usage_stats bundle exec rake db:seed_fu
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
@pshutsin I pulled this branch down to my GDK, generated a GLQL gem via the GLQL branch I'm currently working on, and verified that GLQL generates the right queries for this endpoint, and that the response is expected
For transparency, I used test_queries.rb to run some basic tests. You can pretty much ignore this though, as it's specific to the GQL use-case
Comparing this against CodeSuggestion: end-to-end GLQL aggregation int... (gitlab-org#21212), I can confirm that the endpoint is matching the expected I/O
question (non-blocking): @pshutsin the examples given are all group-level only. I just want to confirm that we expect duoCodeSuggestions to return results for project-level queries too?
Robert Hunt (57251876) at 18 Mar 14:50
fix: remove redundant dimensions parameter from aggregated() call
Robert Hunt (3379b2ca) at 18 Mar 14:35
fix: convert sort parameter to orderBy format for analytics queries
Robert Hunt (aa9ee4b0) at 18 Mar 11:56
fix: update CodeSuggestions GraphQL structure to match backend impl...
Robert Hunt (3be9d601) at 18 Mar 11:25
refactor: revert token match changes
Robert Hunt (71605ec3) at 18 Mar 11:19
feat: add dimension validation for analytics queries
Robert Hunt (2da95ad5) at 18 Mar 09:20
chore: remove continuation guide
Robert Hunt (2a0922f7) at 18 Mar 09:18
feat: implement CodeSuggestions analytics source
... and 22 more commits
This is fine
Robert Hunt (46c1904c) at 17 Mar 23:28
feat: add timeSegment() function support for analytics
... and 3 more commits