Skip to content

generate: make interface consistent with sync and fmt#619

Merged
ErikSchierboom merged 4 commits intoexercism:mainfrom
ee7:generate-consistent-interface
Aug 28, 2024
Merged

generate: make interface consistent with sync and fmt#619
ErikSchierboom merged 4 commits intoexercism:mainfrom
ee7:generate-consistent-interface

Conversation

@ee7
Copy link
Copy Markdown
Member

@ee7 ee7 commented Jun 17, 2022

To-do:

Closes: #334
Closes: #542

@ee7 ee7 requested a review from ErikSchierboom as a code owner June 17, 2022 10:49
@ee7 ee7 marked this pull request as draft June 17, 2022 10:54
@ee7 ee7 force-pushed the generate-consistent-interface branch from 4ee5d51 to 8a64701 Compare June 19, 2022 10:51
@ee7 ee7 force-pushed the generate-consistent-interface branch from 8a64701 to 5e233c7 Compare August 17, 2023 13:00
Comment thread src/cli.nim
Comment thread src/cli.nim
Copy link
Copy Markdown
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.

Good idea

@ErikSchierboom ErikSchierboom force-pushed the generate-consistent-interface branch from 5e233c7 to 0706be0 Compare August 16, 2024 12:17
ee7 and others added 2 commits August 16, 2024 14:17
We must still at least:

- alter `generate.nim`
- update the tests accordingly

Refs: 334
Refs: 542
@ErikSchierboom ErikSchierboom marked this pull request as ready for review August 27, 2024 12:52
@ErikSchierboom ErikSchierboom force-pushed the generate-consistent-interface branch 2 times, most recently from 27d35b4 to bca2698 Compare August 27, 2024 12:56
@ErikSchierboom ErikSchierboom force-pushed the generate-consistent-interface branch from bca2698 to f48ecf4 Compare August 27, 2024 13:00
@ErikSchierboom
Copy link
Copy Markdown
Member

Due to limited time, I've not added new tests but only fixed the existing ones.

@ErikSchierboom ErikSchierboom merged commit 6e74347 into exercism:main Aug 28, 2024
@ee7 ee7 deleted the generate-consistent-interface branch September 5, 2024 19:19
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.

generate: make interface consistent with sync and fmt generate: add -e, --exercise option

2 participants