Skip to content

Relative distance fix #2566

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

Merged
merged 3 commits into from
May 19, 2025
Merged

Relative distance fix #2566

merged 3 commits into from
May 19, 2025

Conversation

mk-mxp
Copy link
Contributor

@mk-mxp mk-mxp commented May 18, 2025

@mk-mxp mk-mxp requested a review from a team as a code owner May 18, 2025 08:28
@@ -1,6 +1,7 @@
# Instructions

Your task is to determine the degree of separation between two individuals in a family tree.
This is similar to the pop culture idea that everyone is [within six degrees of Kevin Bacon][six-bacons].
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't that every actor rather than everyone?
I mean, it might hold true for everyone, but the pop culture idea is about actors.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, I think making it about just actors is a good idea

Copy link
Member

Choose a reason for hiding this comment

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

I’d be in favor as well. The underlying concept is everyone should be up to six degrees of separation from one another, but the Kevin Bacon game is specifically about actors in Hollywood.

Copy link
Member

@ErikSchierboom ErikSchierboom left a comment

Choose a reason for hiding this comment

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

Nice improvement!

@jiegillet jiegillet merged commit 615306d into exercism:main May 19, 2025
7 checks passed
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.

7 participants