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.

Sign up for GitHub

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

Jump to bottom

bob: update cases to reflect more inclusive language #1317

Merged
merged 3 commits into from
Jan 17, 2020
Merged

bob: update cases to reflect more inclusive language #1317

merged 3 commits into from
Jan 17, 2020

Conversation

sebito91
Copy link
Member

This PR updates the bob cases to reflect more inclusive language that before. For details please see: exercism/problem-specifications#1630

@sebito91 sebito91 self-assigned this Jan 17, 2020
@sebito91 sebito91 requested a review from a team January 17, 2020 15:02
ferhatelmas
ferhatelmas suggested changes Jan 17, 2020
View reviewed changes
Copy link
Contributor

@ferhatelmas ferhatelmas left a comment

Choose a reason for hiding this comment

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

version should be bumped too, use generator under .meta folder.

@ferhatelmas ferhatelmas changed the title updating bob cases to reflect more inclusive language bob: update cases to reflect more inclusive language Jan 17, 2020
@ferhatelmas
Copy link
Contributor

@sebito91 FYI, updated PR title, let's try to be consistent (exercise-slug: description)

@sebito91
Copy link
Member Author

Thanks @ferhatelmas will fix that now. Also fixes #1318

@sebito91 sebito91 requested a review from ferhatelmas January 17, 2020 16:06
ferhatelmas
ferhatelmas approved these changes Jan 17, 2020
View reviewed changes
Copy link
Contributor

@ferhatelmas ferhatelmas left a comment

Choose a reason for hiding this comment

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

LGTM

@ferhatelmas ferhatelmas merged commit ae8627e into exercism:master Jan 17, 2020
@ferhatelmas ferhatelmas mentioned this pull request Jan 17, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ferhatelmas ferhatelmas ferhatelmas approved these changes

Assignees

@sebito91 sebito91

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sebito91 @ferhatelmas