exercism / exercism/rust-representer

Replace items not just based on their name

Open
#43 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
6
Forks
10
Avg merge
18h 53m
Merged PRs (30d)
1

Description

I think this test case is inadvertantly exposing a sever limitation of the current representer. This function and its local variable have the same name and are assigned the same placeholder. That should not be the case however, as they refer to different items.

For running the representer in production, this means that two identical solutions may not receive the same representation, simply because two two different items shared or didn't share the same name.

_Originally posted by @senekor in https://github.com/exercism/rust-representer/pull/30#discussion_r1403709927_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.