Skip to content

Acronym#7

Open
TheRealOwenRees wants to merge 6 commits intoexercism:mainfrom
TheRealOwenRees:acronym
Open

Acronym#7
TheRealOwenRees wants to merge 6 commits intoexercism:mainfrom
TheRealOwenRees:acronym

Conversation

@TheRealOwenRees
Copy link
Contributor

Acronym exercise

  • also fixed the adding and checking of testTemplate.js
  • -re-synced older exercise files because of the above fix (would normally do in a separate PR)

@TheRealOwenRees TheRealOwenRees mentioned this pull request Mar 9, 2026
25 tasks
"stevejb71",
"tejasbubane"
],
"files": {
Copy link
Member

Choose a reason for hiding this comment

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

We're missing an editor entry for Acronym.resi here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

how strange, this was automatically added for the other exercises. I will keep my eye on that to see if there is a bug.

@BNAndras BNAndras added x:rep/large Large amount of reputation x:module/practice-exercise Work on Practice Exercises labels Mar 10, 2026
Copy link
Member

Choose a reason for hiding this comment

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

If we're providing interface files, should we expect example solutions to also have one of these in .meta? If so, we'll probably want to copy those interface files over to the src folder when we test the example solutions

Copy link
Contributor Author

Choose a reason for hiding this comment

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

while not mandatory, it will be good to set a standard. I will add one to this exercise, and then create another PR later with fixes for copying the files over when testing

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i have added a PR to fix this, best to merge that before this exercise PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:module/practice-exercise Work on Practice Exercises x:rep/large Large amount of reputation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants