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

Ansible tutorial TODOs from Barcelona (Helena edition) #1831

Merged
merged 18 commits into from
May 28, 2020

Conversation

hexylena
Copy link
Member

@hexylena hexylena commented Mar 4, 2020

xref #1809

@hexylena hexylena added the admin label Mar 4, 2020
@hexylena hexylena added this to the barcelona admin training milestone Mar 4, 2020
@hexylena hexylena requested a review from a team as a code owner March 4, 2020 18:12
@hexylena
Copy link
Member Author

hexylena commented Mar 6, 2020

@natefoo can you review?

natefoo
natefoo previously approved these changes Mar 6, 2020
topics/admin/tutorials/interactive-tools/tutorial.md Outdated Show resolved Hide resolved
topics/admin/tutorials/job-metrics/tutorial.md Outdated Show resolved Hide resolved
topics/admin/tutorials/job-metrics/tutorial.md Outdated Show resolved Hide resolved
topics/admin/tutorials/job-metrics/tutorial.md Outdated Show resolved Hide resolved
@hexylena
Copy link
Member Author

Ready for review if anyone has time today @galaxyproject/training-admin

@hexylena
Copy link
Member Author

@TKlingstrom hey since you're working on admin stuff, let me know if you have any comments on this?

@shiltemann shiltemann merged commit 9d06f2d into galaxyproject:master May 28, 2020
@shiltemann
Copy link
Member

thanks @hexylena! and thanks for the review @natefoo @Slugger70 @nsoranzo and @TKlingstrom!

@@ -296,128 +296,7 @@

---

# Galaxy Dependency Resolver Configuration
Copy link
Member

Choose a reason for hiding this comment

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

@hexylena The text removed here contained more (I think useful) material than what's in topics/admin/tutorials/toolshed/slides.html . Also I've been thinking what's the more appropriate slide deck to include such info about dependency resolution, and I feel it should be in topics/admin/tutorials/tool-management/slides.html (there was one slide, which I removed in d5ba4aa 😊).

What do you think? I can do the work if you agree.

@hexylena
Copy link
Member Author

hexylena commented May 29, 2020 via email

nsoranzo added a commit to nsoranzo/training-material that referenced this pull request Jun 12, 2020
- Move Tool Dependencies slides from "Tool Shed" to "Tool Management"
- Restore and update slides deleted in
  galaxyproject#1831
- Change slide deck and tutorial title to "Galaxy Tool Management with Ephemeris"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants