-
Notifications
You must be signed in to change notification settings - Fork 15
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
Spec about the floor-plan #154
Comments
sorry for the experience. I totally agree. The problem is floor plan is quite big, and I've split the processing pipeline into quite a few smaller functions. I guess it will work with spec2's |
I am totally okay to wait for spec2, as I am waiting for spec2 too. |
New public api has been merged to master. I found a way around with spec1. The public api is simpler, floor-plan now becomes an internal thing. |
to give feedback on errors, guardrails will be used. #87 |
I hit several times that, making change to floor plan cause a lot of un-obvious exceptions. like
emitter
is mandatory, some options if defined, it must have at least one item. I think a spec maybe useful in this case, make the user experience much better.The text was updated successfully, but these errors were encountered: