Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

java: Add javadoc to messages #124

Merged
merged 12 commits into from
Dec 16, 2022
Merged

java: Add javadoc to messages #124

merged 12 commits into from
Dec 16, 2022

Conversation

mpkorstanje
Copy link
Contributor

@mpkorstanje mpkorstanje commented Dec 16, 2022

🤔 What's changed?

Add Javadoc to generated messages

⚡️ What's your motivation?

So people can understand what each field in a message means.

🏷️ What kind of change is this?

  • ⚡ New feature (non-breaking change which adds new behaviour)

📋 Checklist:

  • I agree to respect and uphold the Cucumber Community Code of Conduct
  • I've changed the behaviour of the code
    • I have added/updated tests to cover my changes.
  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.
  • Users should know about my change
    • I have added an entry to the "Unreleased" section of the CHANGELOG, linking to this pull request.

This text was originally generated from a template, then edited by hand. You can modify the template here.

@mpkorstanje mpkorstanje changed the title Add javadoc to messages java: Add javadoc to messages Dec 16, 2022
@mpkorstanje mpkorstanje force-pushed the add-javadoc-to-messages branch from 928d561 to 118a654 Compare December 16, 2022 17:06
@mpkorstanje mpkorstanje force-pushed the add-javadoc-to-messages branch from 118a654 to 4a0824a Compare December 16, 2022 17:10
@mpkorstanje mpkorstanje marked this pull request as ready for review December 16, 2022 17:18
@mpkorstanje mpkorstanje merged commit e2d81b1 into main Dec 16, 2022
@mpkorstanje mpkorstanje deleted the add-javadoc-to-messages branch December 16, 2022 17:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant