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

Launch Red track #64

Closed
loziniak opened this issue Aug 13, 2022 · 7 comments · Fixed by #68
Closed

Launch Red track #64

loziniak opened this issue Aug 13, 2022 · 7 comments · Fixed by #68

Comments

@loziniak
Copy link
Member

It seems we have everything ready, except #48 - Continuous Integration. @ErikSchierboom said he can help with scripts and GitHub Actions, once we implement automated testing of all exercises at once.

@ErikSchierboom
Copy link
Member

PR opened

@loziniak
Copy link
Member Author

Now, that PR #65 is merged, next steps to lanuch can be taken.

@loziniak
Copy link
Member Author

@ErikSchierboom what's next?

@ErikSchierboom
Copy link
Member

I'm testing the track right now!

@ErikSchierboom
Copy link
Member

While not a hard requirement, a nice improvement might be to customize the snippet extractor to skip over comments and such. See https://exercism.org/docs/building/tooling/snippet-extractor and https://github.com/exercism/snippet-extractor/#add-your-language.

@ErikSchierboom
Copy link
Member

ErikSchierboom commented Aug 18, 2022

Another would be to support correct line counting. For that, we use the tokei library, which unfortunately doesn't support red at the moment: https://github.com/XAMPPRocky/tokei#supported-languages Adding a language should not be that hard: https://github.com/XAMPPRocky/tokei/blob/master/CONTRIBUTING.md Fun fact: the recently launched unison track did this: XAMPPRocky/tokei#920

Once the PR has been merged, let me know.

@ErikSchierboom
Copy link
Member

Issues created: #66 and #67

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 a pull request may close this issue.

2 participants