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

Add test generator to track tooling #495

Merged
merged 4 commits into from
Apr 16, 2024

Conversation

mk-mxp
Copy link
Contributor

@mk-mxp mk-mxp commented Apr 13, 2024

As discussed in the forum.

This my first approach to documenting the test generator information for future contributors. I am very open for improvements, be it language, structure or the place it should live in the docs.

@ErikSchierboom

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.

Thanks for this! It's quite a big document, so I think the best approach is to merge this and then I'll double some tweaks in a follow-up PR. The only thing this requires to be added is to have a new entry in https://github.com/exercism/docs/blob/main/building/config.json.

@mk-mxp mk-mxp requested a review from ErikSchierboom April 16, 2024 08:06
@ErikSchierboom ErikSchierboom merged commit 46b3043 into exercism:main Apr 16, 2024
1 check passed
@ErikSchierboom
Copy link
Member

Thanks!

@ErikSchierboom
Copy link
Member

Some tracks also create tests for concept exercises from a similar track-owned data source.

@mk-mxp What tracks do this?

@mk-mxp
Copy link
Contributor Author

mk-mxp commented Apr 16, 2024

Some tracks also create tests for concept exercises from a similar track-owned data source.

@mk-mxp What tracks do this?

I think I mixed it up in my memory from:

Should I remove that or make more into a suggestion: "Tracks may also create tests for concept exercises from a similar track-owned data source."?

@mk-mxp mk-mxp deleted the add-test-generator branch April 16, 2024 11:06
@ErikSchierboom
Copy link
Member

I'll remove it in a future PR :)

@BNAndras
Copy link
Member

Each language may have its own Test Generator, written in that language. should be updated in a future PR to something like Each language may have its own Test Generator, often written in that language.

As written, I interpret that as being a requirement that a test generator be written in the same language as a track. Some tracks may choose not to do this for a variety of reasons.

@ErikSchierboom
Copy link
Member

I'm working on a revised version

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