LSSTDESC / beavis-ci

Enable occasional integration and testing. Like travis-ci but dumber.
BSD 3-Clause "New" or "Revised" License
2 stars 2 forks source link

remove default value for repo? #1

Closed yymao closed 6 years ago

yymao commented 6 years ago

@drphilmarshall I see that you have changed the default repo to

repo=$( git config --get remote.origin.url | cut -d':' -f2 | cut -d'.' -f1 )

I wonder if we should just remove default repo value (i.e., make it a required input).

The workflow I have in my mind is that people would clone this repo, not within but outwith any repos they may want to render, and run like

.beavis-ci LSSTDESC/DC2-analysis
.beavis-ci LSSTDESC/DC2-production
.beavis-ci LSSTDESC/...

I think that's cleaner way to use this tool now that it is standalone. Thoughts?

drphilmarshall commented 6 years ago

Sure! I like that. I just completed all the initial commits (including LICENSE and README) so let's start improving it :-) I'll take this one on right away.