-
Notifications
You must be signed in to change notification settings - Fork 4
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
Episode 5: Content and Structure Review #3
Comments
interesting why GH only allows one assignee per issue /assign @durandom |
I thought mine was E4? Not that the number matters, but to keep our numbers straight : ) |
@isabelizimm good catch, I think our internal and external number are off..I'll change this to ep5 : ) |
@MichaelClifford reviewed and mentioned that the change you are making is not a bug and it is meant to use Ceph because of Elyra pipelines, I proposed to add a description on the env variable or to state at the beginning that AWS credentials and URLs need to be stated for this notebook to work, unless you set that env variable to False (good for development but cannot be done for release as Elyra pipelines require Ceph) |
@goern it seems that's a limitation for private repos on unpaid plans.. 🤷 |
@tumido oh, thats the last thing I envisioned.... |
@goern any reason we want this repo to be private? |
not really, I just didnt see why it needed to be public. feel free to reconfigure it |
@durandom @goern @pacospace @isabelizimm has everyone had their chance to review? Can I move onto finalizing the script? |
The flow looks logical to me! |
/lgtm |
I reviewed. Looks good |
Great! thanks all. Closing this issue since signed off by all. 👍 |
Here is the rough outline of the content I am planning to share in Episode 4.
Please review the document for content, information and overall flow. Once this issue is signed off, I will start on the final script.
The text was updated successfully, but these errors were encountered: