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

Update testing exercise #12

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update testing exercise #12

wants to merge 3 commits into from

Conversation

lucywyman
Copy link
Contributor

@ElijahCaine @tschuy Actually putting effort into this.


.. code-block:: none

$ git clone git@github.com:devopsbootcamp/systemview.git
[email protected]:ElijahCaine/travis-demo.git
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this my personal repo?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I can't fork it to dobc because permissions. Should I use mythmons instead? Or do a different repository? Or omit the option to clone the repo?

@pop
Copy link
Contributor

pop commented Oct 23, 2015

@lucywyman this is pretty solid. I would like to request the following changes

TODO:

  • Change the repo students fork to be the tinsy-flask-app devopsbootcamp repo. It's set up to have tests and has travis ci integration already.
  • Instruct students how to fork, or at least link them to how to fork a repo.
  • Instruct students on how to setup travis CI. I don't recall if forking a travis-setup repo auto-sets up travis or not. Might be good to just include the 'getting started' link.
  • Tell students to look at the yaml file, don't create it. It already exists. Setting up travis on your own for the first time is a hassle, looking at an already setup travis environment is easier when we can't troubleshoot with them.
  • Mention that you have to push to a repo for it to run the first batch of tests (so they should make a change, commit, then push to see the tests run).

Edit: Maybe also link to this: https://travis-ci.org/DevOpsBootcamp/tinsy-flask-app

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