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
feat: Bind modules/<dep_name>/public.ts
to Module.<depName>
#265
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @Blckbrry-Pi and the rest of your teammates on Graphite |
modules/<depname>/public.ts
to Public.<depname>
modules/<depname>/public.ts
to Public.<depname>
a6b4195
to
10e6653
Compare
modules/<depname>/public.ts
to Public.<depname>
modules/<dep_name>/public.ts
to Module.<depName>
10e6653
to
b01d7fe
Compare
There was a problem hiding this 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/mod.ts
Outdated
else return formatPath({ ...parsed, dir: `.` }); | ||
} | ||
|
||
export class GeneratedFile { |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
b20acf1
to
c1a64ac
Compare
c1a64ac
to
75141cc
Compare
3dc84d3
to
f641967
Compare
92767b2
to
e88d846
Compare
f641967
to
a360312
Compare
e88d846
to
a25b912
Compare
Merge activity
|
a360312
to
69a5619
Compare
a25b912
to
44b0347
Compare
No description provided.