-
Notifications
You must be signed in to change notification settings - Fork 561
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
docs: Add multi-lang name overrides page #7429
docs: Add multi-lang name overrides page #7429
Conversation
Signed-off-by: Vikram Vaswani <vikram@dagger.io>
6beee28
to
43aac09
Compare
@@ -0,0 +1,33 @@ | |||
import { dag, Container, object, func, field } from "@dagger.io/dagger" |
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.
@TomChv please review this code
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.
👆 @TomChv
@object() | ||
class MyModule { | ||
/** | ||
* Definition |
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.
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.
Yes because in one case, def_ is a field, in the other he's an argument of the constructor.
It's 2 different things
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.
Sorry, I still didn't understand this. Are the TypeScript and Python snippets equivalent?
- If yes, then why is the additional inline doc required in the TypeScript code?
- Or if not, can you please suggest code changes inline to make them equivalent?
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.
Signed-off-by: Vikram Vaswani <vikram@dagger.io>
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.
Lgtm
We should not document this. It's not clear that it's a good pattern. Let's remove instead. |
No description provided.