cynkra / constructive

Display Idiomatic Code to Construct Most R Objects
https://cynkra.github.io/constructive
Other
131 stars 6 forks source link

feat: show a console message when using construct_clip() #487

Open wurli opened 2 months ago

wurli commented 2 months ago

Closes #486

NB, I had some test failures when running locally, but I think they must be due to differences with my setup because they were all about where linebreaks are placed. Here's an example:

Failure ([test-s3-R6ClassGenerator.R:2:3](vscode-file://vscode-app/Applications/Positron.app/Contents/Resources/app/out/vs/code/electron-sandbox/workbench/workbench.html#)): R6Class
Snapshot of code has changed:
     old                                                        | new                                                  
[54]     ),                                                     |     ),                                           [54]
[55]     private = list(                                        |     private = list(                              [55]
[56]       queue = list(),                                      |       queue = list(),                            [56]
[57]       length = (function() base::length(private$queue)) |> -       length = (function() base::length(private$ [57]
                                                                -         queue)) |>                               [58]
[58]         (`environment<-`)(                                 |         (`environment<-`)(                       [59]
[59]           constructive::.env(                              |           constructive::.env(                    [60]
[60]             "0x123456789",                                 |             "0x123456789",                       [61]

I assume these won't show up in the CI tests. I'll investigate further when I get a chance.

aviator-app[bot] commented 2 months ago

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes. Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This pull request is currently open (not queued).

How to merge

To merge this PR, comment /aviator merge or add the mergequeue label.


See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.
moodymudskipper commented 2 months ago

Thanks! just a minor adjustment and it's good to go!

wurli commented 2 months ago

Nice :) What's the adjustment? One for me to make?