Skip to content

R6 Class - Enhancement #10

@The-Dub

Description

@The-Dub

I was looking at the source code of harbor to figure out why splashr wasn't working and - I don't know if it's a good idea or not, but throwing it out there - I was wondering if it would be easier for debug & sanity check within functions to use R6 classes?

The structure for opening a docker would be something like boat <- harbor$new('NameOfTheDocker')

Other packages (let's say harbor) having dependency on harbor would also be R6 class being sub-class of the harbor class, for example:

splash <- R6Class('splash', 
inheritance = harbor::harbor, 
public = list(
initialize =  function(host = harbor::localhost) {
  harbor::docker_run(host,
                     image = "scrapinghub/splash",
                     detach = TRUE,
                     docker_opts = c("-p", "5023:5023",
                                     "-p", "8050:8050",
                                     "-p", "8051:8051"))
},
...
)

I forked the repo and will try to get a crack at it tonight, but if @hrbrmstr think it's a bad idea, I'll just let it go :) !

Cheers

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions