Stephen Crosby

Results 85 comments of Stephen Crosby

All my mutations do inherit from GraphQL::Schema::Mutation, so it sounds like the first part would have me covered.

Build is failing because it requires https://github.com/drone/drone-runtime/pull/75

As a followup, I ran tests this morning and verified steps that use `ipc: host` actually result in containers with `IpcMode: "host"`. So this is working the way I expect.

I implemented @emkookmer's suggestion in #3392

I'm happy to take on the rubocop stuff if needed. I'm partial to a few of our rules, although most of standardrb is good with me.

I haven't forgotten, but I'm not finding the time to give this the attention it deserves. @burnettk do you have the capacity to review this?

I have one single comment regarding Dynamic attributes. I'd love to see if we can get rid of that, or track down why it was needed, and I also put...

I don't see an issue yet. Can you grab the latest from github and paste the output of `docker build .`?

Stange. You can see that it does work as GitHub does the build for us: https://github.com/errbit/errbit/actions/runs/3252786181/jobs/5339376809 I wonder if there's something about the way your docker daemon is configured. Maybe...

That's really odd. Right there on step 6, it says /root is not writable. Can you run that specific container and debug it? `docker run --rm -it cef1e71b45e1 sh` From...