Skip to content
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

feat: Bind modules/<dep_name>/public.ts to Module.<depName> #265

Merged

Conversation

Blckbrry-Pi
Copy link
Collaborator

No description provided.

@Blckbrry-Pi Blckbrry-Pi changed the title Bind modules/<depname>/public.ts to Public.<depname> feat: Bind modules/<depname>/public.ts to Public.<depname> Apr 24, 2024
@Blckbrry-Pi Blckbrry-Pi force-pushed the 04-24-bind_modules__depname__public.ts_to_public._depname_ branch 4 times, most recently from a6b4195 to 10e6653 Compare April 30, 2024 19:43
@Blckbrry-Pi Blckbrry-Pi changed the title feat: Bind modules/<depname>/public.ts to Public.<depname> feat: Bind modules/<dep_name>/public.ts to Module.<depName> Apr 30, 2024
@Blckbrry-Pi Blckbrry-Pi marked this pull request as ready for review May 1, 2024 13:43
Base automatically changed from 04-17-fix_issue_with_adaptername_conflict_in_generated_prisma_schemas to main May 1, 2024 19:50
@Blckbrry-Pi Blckbrry-Pi force-pushed the 04-24-bind_modules__depname__public.ts_to_public._depname_ branch from 10e6653 to b01d7fe Compare May 1, 2024 20:35
Copy link
Member

@NathanFlurry NathanFlurry left a comment

Choose a reason for hiding this comment

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

need to include an example in the test repo for this too

src/build/gen/public_utils.ts Outdated Show resolved Hide resolved
src/project/module.ts Outdated Show resolved Hide resolved
else return formatPath({ ...parsed, dir: `.` });
}

export class GeneratedFile {
Copy link
Member

Choose a reason for hiding this comment

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

nit: move to separate file

nit: rename to something like GeneratedCodeBuilder or something with "Builder" in the name

needs docs on what this class does

cool helper though

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Moved and renamed, will add jsdoc comments

src/build/gen/public_utils.ts Outdated Show resolved Hide resolved
@Blckbrry-Pi Blckbrry-Pi force-pushed the 04-24-bind_modules__depname__public.ts_to_public._depname_ branch 2 times, most recently from b20acf1 to c1a64ac Compare May 16, 2024 19:58
@Blckbrry-Pi Blckbrry-Pi changed the base branch from main to 05-15-refactor_rewrite_buildregistryproxy_and_comment_src_runtime_proxy.ts_to_make_it_more_understandable_by_maintainers_and_contributors May 16, 2024 19:58
@Blckbrry-Pi Blckbrry-Pi force-pushed the 04-24-bind_modules__depname__public.ts_to_public._depname_ branch from c1a64ac to 75141cc Compare May 17, 2024 12:17
@Blckbrry-Pi Blckbrry-Pi changed the base branch from 05-15-refactor_rewrite_buildregistryproxy_and_comment_src_runtime_proxy.ts_to_make_it_more_understandable_by_maintainers_and_contributors to 05-17-refactor_rewrite_all_codegen_using_the_new_generatedcodebuilder_class_move_everything_to_._src_build_gen_ May 17, 2024 12:17
@Blckbrry-Pi Blckbrry-Pi force-pushed the 05-17-refactor_rewrite_all_codegen_using_the_new_generatedcodebuilder_class_move_everything_to_._src_build_gen_ branch from 3dc84d3 to f641967 Compare May 17, 2024 12:20
@Blckbrry-Pi Blckbrry-Pi force-pushed the 04-24-bind_modules__depname__public.ts_to_public._depname_ branch 2 times, most recently from 92767b2 to e88d846 Compare May 17, 2024 13:04
@NathanFlurry NathanFlurry force-pushed the 05-17-refactor_rewrite_all_codegen_using_the_new_generatedcodebuilder_class_move_everything_to_._src_build_gen_ branch from f641967 to a360312 Compare May 18, 2024 21:00
@NathanFlurry NathanFlurry force-pushed the 04-24-bind_modules__depname__public.ts_to_public._depname_ branch from e88d846 to a25b912 Compare May 18, 2024 21:00
Copy link
Member

NathanFlurry commented May 18, 2024

Merge activity

  • May 18, 5:00 PM EDT: @NathanFlurry started a stack merge that includes this pull request via Graphite.
  • May 18, 5:04 PM EDT: Graphite rebased this pull request as part of a merge.
  • May 18, 5:05 PM EDT: @NathanFlurry merged this pull request with Graphite.

@NathanFlurry NathanFlurry force-pushed the 05-17-refactor_rewrite_all_codegen_using_the_new_generatedcodebuilder_class_move_everything_to_._src_build_gen_ branch from a360312 to 69a5619 Compare May 18, 2024 21:01
Base automatically changed from 05-17-refactor_rewrite_all_codegen_using_the_new_generatedcodebuilder_class_move_everything_to_._src_build_gen_ to main May 18, 2024 21:03
@NathanFlurry NathanFlurry force-pushed the 04-24-bind_modules__depname__public.ts_to_public._depname_ branch from a25b912 to 44b0347 Compare May 18, 2024 21:03
@NathanFlurry NathanFlurry merged commit de289ad into main May 18, 2024
4 checks passed
@NathanFlurry NathanFlurry deleted the 04-24-bind_modules__depname__public.ts_to_public._depname_ branch May 18, 2024 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants