mbutterick
/
describe
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Labels Milestones

New Pull Request

Replace planet requires with local requires of main.rkt. #1

Merged
greghendershott merged 1 commits from master into master 10 years ago
Conversation 2 Commits 1 Files Changed 4

1 Commits (master)

Author SHA1 Message Date
Greg Hendershott e79a2cc397 Replace planet requires with local requires of main.rkt. 
Otherwise this package's implementation would be using the planet
package -- and causing users of this package to download the planet
package -- which doesn't seem right.

Furthermore, my understanding is that it's preferable within a package
to use a relative require -- like `(require "main.rkt")` -- instead of
requiring the collection itself -- like either of `(require describe)`
or `(require (planet williams/describe/describe))`. So even if this were
still a planet package, I think this would be a good change to make.
 10 years ago