Joe Eli McIlvain

Results 330 comments of Joe Eli McIlvain

My preference as expressed in the sync call, is that I think we should fix the syntax highlighter that renders the tutorial before updating the tutorial

Here's the pygments lexer for Pony which needs to be updated: https://github.com/pygments/pygments/blob/master/pygments/lexers/pony.py

Agreed about leaving internal use of the term "bundle".

I can see both sides of the argument, and I'd be happy with either solution. If we put the mention of the `debug` package somewhere else, I just think we...

Hmm... Probably `ifdef`, `compile_error`, and `compile_intrinsic` should all go in a new chapter named something like "Compile-Time Conditionals". The `iftype` keyword would also go in this chapter too when it...

The other ones I mentioned were `compile_error` and `compile_intrinsic`. For `compile_error`, it would make sense to mention that in conjunction with `ifdef`. For `compile_intrinsic`, it's probably just worth mentioning what...

Maybe it could be an integration test that sets up an example `bundle.json` that clones its own repo (`pony-stable`), and imports some types to prove that it worked?

@sporto - I recommend having a subpackage for your tests. I use this pattern on all my packages, and you can see an example of it here: https://github.com/jemc/pony-crdt/tree/master/crdt/test

I mentioned in the sync call today, that I think this change actually makes things somewhat worse with respect to being able to prevent accidental duplication of the dependencies That...

This is definitely a controversial point, and I agree the docs could/should be expanded so we can make our case.