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

Initial setup #9

Closed
wants to merge 7 commits into from
Closed

Initial setup #9

wants to merge 7 commits into from

Conversation

Jbarget
Copy link
Member

@Jbarget Jbarget commented Mar 15, 2017

adds:

  • skeleton server (after running mix phoenix.new)
  • adds travis.yml

@nelsonic
Copy link
Member

Guys I ❤️ your enthusiasm! 🎉
however the whole point of version control is that we can create a completely new version of the app in the existing repository ...
Please simply push this initial-setup branch to the existing git remote: git@github.com:dwyl/tudo.git

also the first part of the mini project is writing a script to download all issues from all repos in an organisation or user as discussed in: dwyl/tudo#146 (comment)
the script already has a repo: https://github.com/dwyl/github-backup

@des-des
Copy link
Member

des-des commented Mar 15, 2017

@nelsonic Thanks for catching this quickly!

We were following what was done for the skills-wheel project. Will push somewhere and fix issue numbers in commit messages.

Sorry, we misunderstood you in that comment.

Do you want us to publish a hex module for this first part? Need to do some more research plus scope out some more setup time.

We have some options here.

  1. Build a script that takes an api token + org name and returns all the issues to caller.
  2. Build a script that does the above and saves to fs. Then, copy / repush that project to tudo and modify it to meet the needs of this project.

@codecov-io
Copy link

codecov-io commented Mar 16, 2017

Codecov Report

❗ No coverage uploaded for pull request base (master@eb3540e). Click here to learn what that means.
The diff coverage is 58.82%.

@@            Coverage Diff            @@
##             master       #9   +/-   ##
=========================================
  Coverage          ?   58.82%           
=========================================
  Files             ?        6           
  Lines             ?       17           
  Branches          ?        0           
=========================================
  Hits              ?       10           
  Misses            ?        7           
  Partials          ?        0
Impacted Files Coverage Δ
web/views/error_helpers.ex 0% <0%> (ø)
web/controllers/page_controller.ex 100% <100%> (ø)
web/views/error_view.ex 100% <100%> (ø)
test/support/conn_case.ex 100% <100%> (ø)
web/router.ex 50% <50%> (ø)
lib/tudo.ex 75% <75%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eb3540e...006a0d2. Read the comment docs.

@des-des
Copy link
Member

des-des commented Mar 16, 2017

@nelsonic

  1. Not sure exactly how we will share code between this project and gh-backup.
  2. Still not sure about the scope for gh-backup (ux, etc).

So we are not sitting on our hands while working out the answers to these questions, would it be possible to continue with this project for now?
I am sure we can easily transpose what we have learned / built here into gh-backup without much hassle once we know what it requires..

@nelsonic
Copy link
Member

@des-des raise questions about https://github.com/dwyl/github-backup
in https://github.com/dwyl/github-backup/issues so others know where to look.

I apologise if something is unclear but a PR is not the place to clarify it
(just so we can keep track of questions/answers in a logical way...)

@Jbarget
Copy link
Member Author

Jbarget commented May 16, 2017

closing as all has moved to https://github.com/dwyl/tudo

@Jbarget Jbarget closed this May 16, 2017
@nelsonic
Copy link
Member

thanks JB! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants