Skip to content

Add 'Include' header to all README files. #8

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.

Sign up for GitHub

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

Jump to bottom
Closed
GitBruno opened this issue Aug 15, 2018 · 1 comment
Closed

Add 'Include' header to all README files. #8

GitBruno opened this issue Aug 15, 2018 · 1 comment
Assignees
@GitBruno
Labels
high-priority Should be done next

Comments

@GitBruno
Copy link

See #7
@GitBruno GitBruno self-assigned this Aug 15, 2018
@GitBruno GitBruno added to do high-priority Should be done next labels Aug 15, 2018
@GitBruno
Copy link
Author

GitBruno commented Aug 17, 2018 •
edited
Loading

No, we should not do this. We need to add a #load header between #install and #use in all README files. It is easy to forget to add the include line to your script after installing a peer. Maybe there is a nice way to prepend installed ExtendScript peers automatically. But at the moment I am not sure what would be a good pattern for that.

@GitBruno GitBruno changed the title Update all patches to use init. Add 'Load' or Use header to all README files. Aug 17, 2018
@GitBruno GitBruno changed the title Add 'Load' or Use header to all README files. Add 'Load' header to all README files. Aug 17, 2018
@GitBruno GitBruno changed the title Add 'Load' header to all README files. Add 'Include' header to all README files. Aug 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GitBruno GitBruno

Labels
high-priority Should be done next
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@GitBruno