Skip to content

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

Merged
merged 7 commits into from
Mar 23, 2015
Merged

Script Libraries blog post #2

merged 7 commits into from
Mar 23, 2015

Conversation

paulbouwer
Copy link
Contributor

No description provided.

@adamralph
Copy link
Contributor

❤️ it!

@glennblock any comments before we merge?

@adamralph adamralph mentioned this pull request Mar 7, 2015
22 tasks
@glennblock
Copy link
Contributor

Awesome post Paul! I have a few small edits.

Also, we shouldn't merge till we release 0.14.

@adamralph
Copy link
Contributor

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).
Copy link
Member

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@jrusbatch
Copy link
Member

Looks great! Nice post!

@paulbouwer
Copy link
Contributor Author

@glennblock Send through the edits and I'll update the post.

@glennblock
Copy link
Contributor

Will do it today
On Sun, Mar 8, 2015 at 3:19 PM Paul Bouwer notifications@github.com wrote:

@glennblock https://github.com/glennblock Send through the edits and
I'll update the post.


Reply to this email directly or view it on GitHub
#2 (comment)
.

@adamralph
Copy link
Contributor

Is this ready to go? The code is ready to be released.

@adamralph
Copy link
Contributor

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.

@paulbouwer
Copy link
Contributor Author

It's good to go from my side. I think @glennblock wanted some edits to the
post that still need to be made.

On Sunday, March 22, 2015, Adam Ralph notifications@github.com wrote:

Is this ready to go? The code is ready to be released.


Reply to this email directly or view it on GitHub
#2 (comment)
.

adamralph added a commit that referenced this pull request Mar 23, 2015
Script Libraries blog post
@adamralph adamralph merged commit 25ab89a into scriptcs:master Mar 23, 2015
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.

4 participants