fix: use parameter generateTimestamp to suppress timestamp in @Genera…#441
fix: use parameter generateTimestamp to suppress timestamp in @Genera…#441anessi wants to merge 1 commit intoasyncapi:masterfrom
Conversation
|
I really like this option but maybe it would be better the current behaviour as default and make it opt-in, especially if it is for a patch release. Also, parameters should be listed in the README. |
86333c2 to
1d9f243
Compare
|
Added a default value for |
|
@anessi Thank you for the contribution! |
annotation asyncapi#433 Signed-off-by: anessi <16045045+anessi@users.noreply.github.com>
|
I added 2 tests. I suggest to update the README.md with the following notes:
This will help Java devs that are not very familiar with Node. |
|
This pull request has been automatically marked as stale because it has not had recent activity 😴 It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation. There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model. Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here. Thank you for your patience ❤️ |
|
This is still an issue. |
Description
Use parameter
generateTimestampto suppress timestamp in@Generatedannotation, see #433Related issue(s)
Fixes #433