Skip to content

[Acronym] Approach docs cleanup#4258

Open
Yrahcaz7 wants to merge 5 commits into
exercism:mainfrom
Yrahcaz7:acronym-approach-cleanup
Open

[Acronym] Approach docs cleanup#4258
Yrahcaz7 wants to merge 5 commits into
exercism:mainfrom
Yrahcaz7:acronym-approach-cleanup

Conversation

@Yrahcaz7

@Yrahcaz7 Yrahcaz7 commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

As discussed in issue #4197.

@Yrahcaz7 Yrahcaz7 changed the title Acronym approach cleanup [Acronym] Approach docs cleanup Jun 27, 2026
Comment thread exercises/practice/acronym/.approaches/functools-reduce/snippet.txt
@Yrahcaz7

Copy link
Copy Markdown
Contributor Author

For some reason the CI failed... I don't think I modified anything outside the .approaches folder though?

@Yrahcaz7 Yrahcaz7 marked this pull request as ready for review June 27, 2026 03:04
@Yrahcaz7

Copy link
Copy Markdown
Contributor Author

@BethanyG, do you know what's going on with the CI?

@BethanyG

Copy link
Copy Markdown
Member

@Yrahcaz7 -- Yuppers. If you take a look at the housekeeping run, you'll see that when the tests were regenerated for the track, the Bob exercise failed because there were new tests added upstream. I'll regenerate the tests and merge them, and then we can pull the changes into your branch so that the CI will pass.

@BethanyG

Copy link
Copy Markdown
Member

Huh. Usually I can do this from the interface, but I don't see the button. I've merged the changes - can you pull them into your branch? That should fix the CI.

Yrahcaz7 added 5 commits June 27, 2026 03:04
also fix missing links in `introduction.md`
also correct a few statements about performance
also fix a few incorrect things in the regex approaches
also add username to contributors array
@Yrahcaz7 Yrahcaz7 force-pushed the acronym-approach-cleanup branch from cb3683d to 935f65d Compare June 27, 2026 07:05
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