exercism / exercism/go-representer

Minor improvements to the setup

Open
#16 2 comments 1 reaction 0 assignees View on GitHub
Dominant language
Go
Stars
4
Forks
8
Avg merge
8d 6h
Merged PRs (30d)
1

Description

I had a look at the current state and noticed a couple of minor things we might want to address:

* Currently there is no test case that covers the multiple files case
* Is there any reason we call the file with the test fixtures `tests` and not `testdata` which is the official name?
* Would be good to have a .gitignore file that contains `vendor` in case someone runs in vendored mode, the `represent` executable that is created when running the command from the readme and any `.exe` files that are created on Windows
* Would be good if the readme also showed an example command that would actually run, I found this very helpful with the testrunner, e.g. `./represent hamming ./representer/tests/hamming/1 .` (not sure how it looks for windows)

cc @tehsphinx

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.