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

Move /=== to a Dojo-only affordance. #63

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

sigilante
Copy link
Contributor

No description provided.

@sigilante sigilante marked this pull request as ready for review July 15, 2024 22:59
@tinnus-napbus
Copy link
Contributor

The problem with these is arguably the inverse: that we've lead people to expect our/desk/now due to how it's implemented in the Dojo. As I understand it the basic purpose is to specify the filepath to include in the stacktrace during a compilation failure, and I guess it's exposed to the thing being compiled via % & /=== so that it can either include it in its own error messages or pass it through to things that itself compiles

@sigilante
Copy link
Contributor Author

This is an interesting point—if /=== usage is the result of scope creep, then we need to decide a good partition if we want to "fix" the situation. Certainly /=== is convenient, and I can see how it's useful in stack traces. But /app usage is the real problem, and the /(scot %p our.bowl) etc. pattern is really pretty wretched.

@midden-fabler
Copy link
Contributor

This is an interesting point—if /=== usage is the result of scope creep, then we need to decide a good partition if we want to "fix" the situation. Certainly /=== is convenient, and I can see how it's useful in stack traces. But /app usage is the real problem, and the /(scot %p our.bowl) etc. pattern is really pretty wretched.

(en-beam [our %base da+now] /foo) is nice, and underused.

@sigilante
Copy link
Contributor Author

I've been thinking about this, and I think that the better course is to stop teaching /=== then, and instead teach en-beam.

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.

3 participants