Skip to content
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

Fix ruby code samples to only use keyword arguments and hash dot access #431

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

andrii-balitskyi
Copy link
Contributor

@andrii-balitskyi andrii-balitskyi commented Nov 19, 2024

@andrii-balitskyi andrii-balitskyi requested a review from a team as a code owner November 19, 2024 12:59
@andrii-balitskyi andrii-balitskyi changed the title Fix ruby code samples to only use keyword arguments Fix ruby code samples to only use keyword arguments and hash dot access Nov 19, 2024
@@ -1594,7 +1594,7 @@ if (updatedFrontDoor.can_program_online_access_codes) {
```ruby
# Confirm that the device supports online access codes.
# Here's another capability flag!
if (updated_front_door['can_program_online_access_codes'])
if (updated_front_door.can_program_online_access_codes)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@andrii-balitskyi Did I have this reference to a property wrong before, or what changed to make it possible to refer to a property using dot notation now?

@razor-x
Copy link
Contributor

razor-x commented Nov 19, 2024

@andrii-balitskyi @DebbieAtSeam let's add this to the migration guide as well

Copy link
Contributor

@DebbieAtSeam DebbieAtSeam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that merging this PR will break any Supademos and supported device tables in any edited topics, but that's probably fine because all the Supademos show the old Console design, and all the supported device tables are currently broken anyway.

@razor-x razor-x merged commit c0e9c71 into main Nov 19, 2024
13 checks passed
@razor-x razor-x deleted the fix-ruby-code-samples branch November 19, 2024 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants