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

Rename bracket-push exercise #284

Merged
merged 2 commits into from
Jun 1, 2019
Merged

Conversation

smuroff
Copy link
Contributor

@smuroff smuroff commented May 24, 2019

I had troubles with configlet because bracket-push exercise was renamed (exercism/problem-specifications#1501) . Inspired by #277.

$ configlet generate cpp/
-> 1 error occurred:

* open D:\develop\github\exercism\problem-specifications\exercises\bracket-push\metadata.yml: The system cannot find the path specified.

Closes #277

@arcuru
Copy link
Contributor

arcuru commented Jun 1, 2019

Ah, I was hoping to try to get #276 merged first to reduce churn, but if you're getting hit with this it should be fixed. Thanks!

Also FYI, you can auto-close related issues using https://help.github.com/en/articles/closing-issues-using-keywords if you put the message in the first note in the PR :)

This is failing the CI because of the rename. The Rust track hit the same issue, because I based our script off of theirs: exercism/rust#819

Luckily they already found the fix. I'm going to add their fix here myself and merge, there aren't any other issues as far as I can tell.

@arcuru arcuru merged commit d4facc8 into exercism:master Jun 1, 2019
@smuroff smuroff deleted the rename_bracket_push branch June 1, 2019 18:00
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.

bracket-push: renamed to matching-brackets
2 participants