Skip to content
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

Mono documentation #21

Merged
merged 3 commits into from
Sep 30, 2015
Merged

Mono documentation #21

merged 3 commits into from
Sep 30, 2015

Conversation

arschles
Copy link
Contributor

TODO:

  • test upload and queue steps
  • add a config file, just as in the java examples

cc/ @treeder

- upload and queue steps not tested
- no config file given yet

cc/ @treeder
@arschles arschles changed the title preliminary mono documentation [WIP] preliminary mono documentation Sep 28, 2015
Now run it to test it out:

```sh
docker run --rm -v "$(pwd)":/worker -w /worker iron/mono:4.0.1 sh -c 'mono worker101.exe -payload hello.payload.json -id 123'
Copy link
Contributor

Choose a reason for hiding this comment

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

Change to env vars for payload.

I could have missed it, but I didn’t see support for grabbing a payload
in
https://github.com/iron-io/iron_dotnet/tree/master/src/IronSharp.IronWor
ker. this should do for now

cc/ @treeder
@arschles
Copy link
Contributor Author

@treeder this now uses an env var to get the payload file as requested. none of the client libraries listed in the dev center (http://grcodemonkey.github.io/iron_sharp/, https://github.com/odeits/IronTools and https://github.com/iron-io/iron_dotnet) seem to provide payload lookup so unable right now to follow your request. still combing through the code to make sure but it's a line of code to get the value of an env var in C# anyway.

please let me know if there are any other blockers to merging.

@treeder
Copy link
Contributor

treeder commented Sep 29, 2015

Should update one of the dotnet libs (whichever is the best one) to read the env vars since that's our primary way of doing things now (regardless of language).

@arschles
Copy link
Contributor Author

@treeder it's on my todo list, but rather merge this now then update

@arschles arschles changed the title [WIP] preliminary mono documentation Mono documentation Sep 29, 2015
@treeder
Copy link
Contributor

treeder commented Sep 30, 2015

Ok, that doesn't look too bad like this anyways, couple lines, I like it.

treeder added a commit that referenced this pull request Sep 30, 2015
@treeder treeder merged commit 6fb8f9c into master Sep 30, 2015
@treeder treeder deleted the mono branch October 2, 2015 18:43
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.

2 participants