-
Notifications
You must be signed in to change notification settings - Fork 3
Script Libraries blog post #2
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
Conversation
❤️ it! @glennblock any comments before we merge? |
Awesome post Paul! I have a few small edits. Also, we shouldn't merge till we release 0.14. |
I've updated the checklist at scriptcs/scriptcs#947 to reflect that |
``` | ||
You can see that this script contains basic `Add` and `Subtract` functions. | ||
|
||
It is possible to pull in a traditional Script Pack via the `Require<T>` mechanism. Here we are pulling in the `Logger` Script Pack. Note that you cannot use `var` to declare the variable holding the Script Pack within Script Libraries. You **MUST** use an explictly typed variable. For the background on why, refer to the *Adding a library reference* section of the Script Libraries [Design Document](https://github.com/scriptcs/scriptcs/wiki/Script-Libraries). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should link directly to the section being referred to here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
Looks great! Nice post! |
@glennblock Send through the edits and I'll update the post. |
Will do it today
|
Is this ready to go? The code is ready to be released. |
The date will need changing, so whoever does the merge should pull down Paul's branch locally, add a commit to change the date and then push to upstream master. That will also close this PR. |
It's good to go from my side. I think @glennblock wanted some edits to the On Sunday, March 22, 2015, Adam Ralph notifications@github.com wrote:
|
No description provided.