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

accumulate: move important hint to instructions #1999

Merged
merged 1 commit into from
Dec 7, 2024
Merged

Conversation

senekor
Copy link
Contributor

@senekor senekor commented Dec 6, 2024

No description provided.

Copy link
Contributor

github-actions bot commented Dec 6, 2024

Hello 👋 Thanks for your PR.

This repo does not currently have dedicated maintainers. Our cross-track maintainers team will attempt to review and merge your PR, but it will likely take longer for your PR to be reviewed.

If you enjoy contributing to Exercism and have a track-record of doing so successfully, you might like to become an Exercism maintainer for this track.

Please feel free to ask any questions, or chat to us about anything to do with this PR or the reviewing process on the Exercism forum.

(cc @exercism/cross-track-maintainers)

@senekor senekor force-pushed the senekor/kqmmotlvyols branch from 145c5a0 to 83141f5 Compare December 6, 2024 22:14
@senekor
Copy link
Contributor Author

senekor commented Dec 6, 2024

forum discussion


## Workflow

The tests for this exercise will cause compile time errors,
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The tests for this exercise will cause compile time errors,
The tests for this exercise will cause compile time errors.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The next line is actually part of the sentence, I put it all on one line.

Copy link
Member

@BNAndras BNAndras left a comment

Choose a reason for hiding this comment

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

Looks good. One minor typo.

@senekor senekor force-pushed the senekor/kqmmotlvyols branch from 83141f5 to 6ab348b Compare December 7, 2024 08:45
@senekor senekor requested a review from BNAndras December 7, 2024 08:47
@senekor senekor merged commit 6c32138 into main Dec 7, 2024
10 checks passed
@senekor senekor deleted the senekor/kqmmotlvyols branch December 7, 2024 17:18
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.

2 participants