-
-
Notifications
You must be signed in to change notification settings - Fork 513
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
Update deprecated method - Scenario.embed() to Scenario.attach() "Fixes #486" #550
Conversation
[5.7.0](https://github.com/cucumber/cucumber-jvm/blob/main/CHANGELOG.md#570-2020-05-01) (2020-05-01) Added [Java] Scenario.log(String) & Scenario.attach(byte[], String, String) (#1893 Tim te Beek) Deprecated [Java] Scenario.write(String) & Scenario.embed(byte[], String, String) (#1893 Tim te Beek)
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.
Could you scan the doc for all instances of this in all languages. As it makes sense to fix them all up now (They're all updated).
@luke-hill, thanks for looking into this. I looked through the docs and could find only one other instance of embed, for Ruby. I have updated the code, but I am not familiar with Ruby. So please review the change. 4.0.0.rc.4 |
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 but I have not scanned the project for other instances.
Hi @josepj, Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾 In return for this generous offer we hope you will:
On behalf of the Cucumber core team, |
5.7.0 (2020-05-01)
Added
[Java] Scenario.log(String) & Scenario.attach(byte[], String, String) (#1893 Tim te Beek)
Deprecated
[Java] Scenario.write(String) & Scenario.embed(byte[], String, String) (#1893 Tim te Beek)