Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: add PartialClaudeCodeConfig type for better configuration handling #115

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

Conversation

Copy link
Member

@ThomasK33 ThomasK33 commented Aug 8, 2025

Introduce PartialClaudeCodeConfig type for better type checking

This PR introduces a new PartialClaudeCodeConfig type that extends the existing ClaudeCodeConfig type. The type signature for the setup function and the dev-config have been updated to use this new type, which better represents that configuration options are partial and can be overridden.

This change improves type checking and makes it clearer that the configuration passed to setup() doesn't need to include all possible options.

@ThomasK33 ThomasK33 force-pushed the thomask33/feat_add_on_new_file_reject_option_to_control_empty_buffer_behavior branch from 44647fb to b350b53 Compare August 8, 2025 14:18
@ThomasK33 ThomasK33 force-pushed the thomask33/feat_add_PartialClaudeCodeConfig_type_for_better_configuration_handling branch from 004c32f to 4ccf295 Compare August 8, 2025 14:18
@ThomasK33 ThomasK33 force-pushed the thomask33/feat_add_on_new_file_reject_option_to_control_empty_buffer_behavior branch from b350b53 to 0bcb677 Compare August 8, 2025 19:34
@ThomasK33 ThomasK33 force-pushed the thomask33/feat_add_PartialClaudeCodeConfig_type_for_better_configuration_handling branch from 4ccf295 to cc09df9 Compare August 8, 2025 19:34
@ThomasK33 ThomasK33 force-pushed the thomask33/feat_add_on_new_file_reject_option_to_control_empty_buffer_behavior branch from 0bcb677 to 00cc3d9 Compare August 10, 2025 16:22
@ThomasK33 ThomasK33 force-pushed the thomask33/feat_add_PartialClaudeCodeConfig_type_for_better_configuration_handling branch from cc09df9 to 696920c Compare August 10, 2025 16:22
@ThomasK33 ThomasK33 force-pushed the thomask33/feat_add_on_new_file_reject_option_to_control_empty_buffer_behavior branch from 00cc3d9 to fbe5b80 Compare September 3, 2025 15:16
@ThomasK33 ThomasK33 force-pushed the thomask33/feat_add_PartialClaudeCodeConfig_type_for_better_configuration_handling branch from 696920c to efeff6f Compare September 3, 2025 15:16
Copy link
Member Author

ThomasK33 commented Sep 10, 2025
edited
Loading

Merge activity

  • Sep 10, 3:36 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Sep 10, 3:40 PM UTC: Graphite rebased this pull request as part of a merge.
  • Sep 10, 3:40 PM UTC: @ThomasK33 merged this pull request with Graphite.

@ThomasK33 ThomasK33 changed the base branch from thomask33/feat_add_on_new_file_reject_option_to_control_empty_buffer_behavior to graphite-base/115 September 10, 2025 15:37
@ThomasK33 ThomasK33 changed the base branch from graphite-base/115 to main September 10, 2025 15:38
Change-Id: I7e94c9ebbace3654b1f1cfad8b0e3f3899c786cb
Signed-off-by: Thomas Kosiewski <tk@coder.com>
@ThomasK33 ThomasK33 force-pushed the thomask33/feat_add_PartialClaudeCodeConfig_type_for_better_configuration_handling branch from efeff6f to ea94fe2 Compare September 10, 2025 15:39
@ThomasK33 ThomasK33 merged commit 89fc08e into main Sep 10, 2025
1 of 3 checks passed
@ThomasK33 ThomasK33 deleted the thomask33/feat_add_PartialClaudeCodeConfig_type_for_better_configuration_handling branch September 12, 2025 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

1 participant

AltStyle によって変換されたページ (->オリジナル) /