-
Notifications
You must be signed in to change notification settings - Fork 45
Make this repo public #11
Comments
I guess there's one issue - we should tag a release |
Created first milestone, if there's anything else we need please raise a ticket and attach it to this milestone. |
I'm happy to open this once the PRs are all merged. Your suggestions are all fine - using Travis is no problem, we already use it for many other Cloudflare repos. |
No more outstanding PRs, is everyone happy with making this repo public now? This will allow me to tag a release and complete at least first 2 points in this ticket (docker image isn't critical). |
LGTM 👍 |
@mattbostock any objections? |
No objections from me. Just one question - are we going to use the timestamp format for release tags? IMHO it'd be more useful to use semantic versioning for a public project. |
Also, minor nit, but we should definitely add a screenshot to the README. Maybe using the sample data you created? |
I'm happy with both version schemes, @jamesog you're needed to get a quorum |
Should we use |
Yep, I'd prefix with v. You have your quorum. Let the code be free. |
I also agree with 'v' - I believe that's what GitHub uses by default if you create a tag manually. |
I'm thinking that, rather than v1.0, we make the first version be v0.0.1. That'll allow for changes (without worrying about backwards compatibility) while the code settles a bit. Using SemVer, there's a big difference between pre-1.0.0 and 1.0.0 onwards: |
|
We're done with PRs I think, I'll open the repo and setup travis |
🎉 |
Made a PR for travis - #17 |
will deploy a new build internally and if all goes well we'll tag a release |
Repo is public so closing |
There are a few open PRs, can we open this repo once they are merged?
Are there any other outstanding issues preventing us from making it public?
Small tasks to be done once it's open:
The text was updated successfully, but these errors were encountered: