Comments (16)
good idea @kormoc ,
i would suggest a more nuanced approach though: commit/push directly when you're pretty sure the commit is the right way forward and won't cause complaints, and still file PR's in case of doubt or if something might break compatibility.
from diamond.
Yeah, I love me my peer reviews as well. But we can now review each other's
code
Pada 30 Apr 2014 20:44, "Dieter Plaetinck" [email protected]
menulis:
good idea @kormoc https://github.com/kormoc ,
i would suggest a more nuanced approach though: commit/push directly when
you're pretty sure the commit is the right way forward and won't cause
complaints, and still file PR's in case of doubt or if something might
break compatibility.—
Reply to this email directly or view it on GitHubhttps://github.com//issues/2#issuecomment-41797504
.
from diamond.
btw @kormoc if i were you, i would delete this repo, move brightcoveos/diamond to here, and then fork this back to brightcoveos/diamond; doing it via a move means issues, wiki, watchers, stars, etc get moved properly. and it's more clear which is the authorative upstream version and you can still maintain a fork for brightcove.
from diamond.
+1 on the transfer the repository
from diamond.
Sadly, I just can't :(
I've never worked for brightcove and ever contact I have there is gone. I've been unable to reach the right people to get you folks added directly to the repo as developers and I don't have the permissions to move the repo around. I'll keep trying, but I don't believe things will change.
from diamond.
I don't know how to handle git submodules well enough, but we should move the BrightcoveOS wiki to this repo, no?
from diamond.
@Lacrymology Done. :)
from diamond.
I can be a BC contact. @kormoc -- will contact you outside this issue.
from diamond.
@kormoc - I hope I'm not stealing this thread, but should issues and PRs be re-open against this repo?
Thanks.
from diamond.
@zstyblik you should issue PRs here
from diamond.
Thanks @mzupan, but that's not what I'm asking about. I'm asking whether issues and PRs from https://github.com/BrightcoveOS/Diamond should be re-opened here(and probably closed there).
Of course, I will open new issues and PRs here in the future.
from diamond.
@aronatkins @kormoc news on moving BrightcoveOS/Diamond here including issues/PR/etc as @Dieterbe suggested? that'd make things much more clear on what development to follow
thanks!
from diamond.
@kormoc I still have some contacts there. I'm sending you an email to the address listed on your github profile.
from diamond.
@ccope -- I hooked @kormoc up with Brian F. at Brightcove while I was still there. Brian's one of the BrightcoveOS admins. Not sure if that ever went anywhere. I've just left Brightcove myself, but can help still work to get the repo migrated if that's still needed. Feel free to CC me on anything.
from diamond.
Oh my, we have passing tests again!
from diamond.
@kormoc Thoughts on closing old PRs in the brightcove repo? And maybe changing the readme so that people stop referring to it?
from diamond.
Related Issues (20)
- delete = False in [[FilesCollector]] config file is delete = 'False', so it is True HOT 1
- VM metric collector config question HOT 1
- JolokiaCollector is not working HOT 1
- NetworkCollector sends absurd data to grafana HOT 1
- ERROR: diamond requires Python '==2.7' but the running Python is 2.7.16 HOT 1
- MongoDBCollector ignores metrics_whitelist? HOT 2
- ApcupsdCollector also working with apcaccess binary instead of network
- Insecure TLS in RabbitMQ Collector HOT 1
- mdstat may fail on some _parse_array_member_state situations HOT 2
- Multi-server support for NginxCollector HOT 1
- Problem with GlusterFS HOT 1
- Dependabot couldn't authenticate with https://pypi.python.org/simple/
- MongoDBCollector reduce sending data HOT 1
- diamond fails to run at boot with "Name or service not known" HOT 2
- Spike after missing data points HOT 1
- Error: [MainThread] Took too long to run! Killed! HOT 1
- Unable lo load second handler metrices
- Syntax error HOT 2
- collect_without_dimension not in latest release HOT 1
- "global name 'self' is not defined" in diamond/utils/scheduler.py HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from diamond.