Open rhoegg opened 7 years ago
I think this is a cool idea, probably needs to be done at a hack night virtual or otherwise so that we can make sure it is all working.
I did it at the last one :) I think we just need to merge it.
On Fri, Mar 17, 2017 at 4:31 PM, smurphy8 notifications@github.com wrote:
I think this is a cool idea, probably needs to be done at a hack night virtual or otherwise so that we can make sure it is all working.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/codeforokc/openbudgetokc/pull/122#issuecomment-287476429, or mute the thread https://github.com/notifications/unsubscribe-auth/AACuDw5yCyYpI_0a6AiqQangabPMHU10ks5rmvuxgaJpZM4MN4Ds .
Okay I will merge when I get in tonight.
On Mon, Mar 20, 2017 at 10:03 AM, Ryan Hoegg notifications@github.com wrote:
I did it at the last one :) I think we just need to merge it.
On Fri, Mar 17, 2017 at 4:31 PM, smurphy8 notifications@github.com wrote:
I think this is a cool idea, probably needs to be done at a hack night virtual or otherwise so that we can make sure it is all working.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/codeforokc/openbudgetokc/pull/ 122#issuecomment-287476429, or mute the thread https://github.com/notifications/unsubscribe-auth/AACuDw5yCyYpI_ 0a6AiqQangabPMHU10ks5rmvuxgaJpZM4MN4Ds .
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/codeforokc/openbudgetokc/pull/122#issuecomment-287787291, or mute the thread https://github.com/notifications/unsubscribe-auth/AALJjGnqC93oUR2MnE1Dl2i0JxPEeo1hks5rnpVSgaJpZM4MN4Ds .
-- Scott Murphy scottmurphy09@gmail.com
Yes, sort of? :) Turns out the encrypted token is tied to the repository path, so I couldn't test the exact PR in my own repo. I tested with my own token on a separate branch "ryan-test-build"
https://travis-ci.org/rhoegg/openbudgetokc/branches