Skip to content

chore: move usage types to new package #19103

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 20, 2025
Merged

chore: move usage types to new package #19103

merged 1 commit into from
Aug 20, 2025
+470 −225

Conversation

deansheather
Copy link
Member

No description provided.

@deansheather deansheather mentioned this pull request Jul 30, 2025
Merged
@deansheather Graphite App
Copy link
Member Author

deansheather commented Jul 30, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@deansheather deansheather mentioned this pull request Jul 30, 2025
Merged
@github-actions github-actions bot added the stale This issue is like stale bread. label Aug 7, 2025
@github-actions github-actions bot closed this Aug 10, 2025
@deansheather deansheather reopened this Aug 12, 2025
@deansheather deansheather removed the stale This issue is like stale bread. label Aug 12, 2025
@deansheather deansheather force-pushed the 07-30-chore_move_usage_types_to_new_package branch from 042a5e2 to fb89896 Compare August 15, 2025 09:28
@deansheather deansheather force-pushed the 07-30-chore_wire_up_usage_tracking_for_managed_agents branch from 89b1757 to 397a84d Compare August 15, 2025 09:28
@deansheather deansheather marked this pull request as ready for review August 15, 2025 09:30
@deansheather deansheather force-pushed the 07-30-chore_move_usage_types_to_new_package branch from fb89896 to c23a50f Compare August 15, 2025 14:49
@deansheather deansheather force-pushed the 07-30-chore_wire_up_usage_tracking_for_managed_agents branch from 397a84d to 7865f96 Compare August 15, 2025 14:49
@deansheather deansheather force-pushed the 07-30-chore_move_usage_types_to_new_package branch from c23a50f to bf80953 Compare August 15, 2025 15:08
@deansheather deansheather force-pushed the 07-30-chore_wire_up_usage_tracking_for_managed_agents branch 2 times, most recently from 40d1681 to 7c36cc0 Compare August 15, 2025 15:19
@deansheather deansheather force-pushed the 07-30-chore_move_usage_types_to_new_package branch from bf80953 to 6b2f72f Compare August 15, 2025 15:19
@deansheather deansheather force-pushed the 07-30-chore_wire_up_usage_tracking_for_managed_agents branch from 7c36cc0 to af6661d Compare August 15, 2025 15:31
@deansheather deansheather requested a review from Emyrk as a code owner August 15, 2025 15:31
@deansheather deansheather force-pushed the 07-30-chore_wire_up_usage_tracking_for_managed_agents branch from af6661d to 3aea6d1 Compare August 15, 2025 15:31
@deansheather deansheather force-pushed the 07-30-chore_move_usage_types_to_new_package branch 2 times, most recently from f58d25a to 0d467bb Compare August 15, 2025 17:55
Emyrk
Emyrk approved these changes Aug 18, 2025
View reviewed changes
coderd/usage/usagetypes/events.go
// is validated.
func ParseEvent[T Event](data json.RawMessage) (T, error) {
dec := json.NewDecoder(bytes.NewReader(data))
dec.DisallowUnknownFields()
Copy link
Member

Choose a reason for hiding this comment

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

👍

@deansheather deansheather mentioned this pull request Aug 20, 2025
Merged
@deansheather deansheather force-pushed the 07-30-chore_move_usage_types_to_new_package branch from 0d467bb to b343097 Compare August 20, 2025 13:15
@deansheather deansheather force-pushed the 07-30-chore_wire_up_usage_tracking_for_managed_agents branch from 2d92d73 to 29655e5 Compare August 20, 2025 13:15
@deansheather Graphite App
Copy link
Member Author

deansheather commented Aug 20, 2025 •
edited
Loading

Merge activity

  • Aug 20, 1:37 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Aug 20, 1:40 PM UTC: Graphite rebased this pull request as part of a merge.
  • Aug 20, 1:48 PM UTC: @deansheather merged this pull request with Graphite.

@deansheather deansheather changed the base branch from 07-30-chore_wire_up_usage_tracking_for_managed_agents to graphite-base/19103 August 20, 2025 13:38
@deansheather deansheather changed the base branch from graphite-base/19103 to main August 20, 2025 13:38
@deansheather deansheather requested a review from aslilac as a code owner August 20, 2025 13:38
@deansheather deansheather force-pushed the 07-30-chore_move_usage_types_to_new_package branch from b343097 to e0be548 Compare August 20, 2025 13:39
@deansheather deansheather merged commit 1a601c3 into main Aug 20, 2025
32 checks passed
@deansheather deansheather deleted the 07-30-chore_move_usage_types_to_new_package branch August 20, 2025 13:48
@github-actions github-actions bot locked and limited conversation to collaborators Aug 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@spikecurtis spikecurtis spikecurtis approved these changes

@Emyrk Emyrk Emyrk approved these changes

@aslilac aslilac Awaiting requested review from aslilac

Assignees

@deansheather deansheather

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@deansheather @spikecurtis @Emyrk