-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
bracket-push: Add new exercise; README/test-exercise: no hyphens in module names #13
Conversation
Ceylon doesn't tend to like these: ``` $ ceylon compile bracket-push ceylon compile: Invalid module name or source file: bracket-push Module names should not contain any version part. ``` Despite the fact that I didn't give a version part, I think the dash causes it to be interpreted that way. It works if I make it bracketpush, so I'll just do that. I looked at https://herd.ceylon-lang.org and while some modules there have dashes in the name, they are all the "Legacy Jar" format, so I imagine this is not replicable.
As discovered (see README change removing dash from module name), module names with dashes don't seem to be accepted.
e732d49
to
a8c40e7
Compare
if (opens.contains(c)) { | ||
expectedOpens.push(c); | ||
} else if (exists open = openFor[c]) { | ||
if (exists last = expectedOpens.pop(), last == open) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
empty if
makes me uncomfortable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, using expectedOpens.pop() else '!'
@@ -0,0 +1,4 @@ | |||
module bracketpush "1.0" { | |||
import "ceylon.collection" "1.3.1"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move to example dir?
a8c40e7
to
416b559
Compare
416b559
to
6954c7d
Compare
Student's only has ceylon.test, example's has whatever it needs. This pattern has already been used in sieve (exercism#16) and bracket-push (exercism#13) anyway, so it needs to be documented.
No description provided.