-
Notifications
You must be signed in to change notification settings - Fork 79
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FR: create a .rscIgnore files to ignore files from being bundled #368
Comments
Linked issue in the |
FWIW, (As an intermediate step, |
Wouldn't that just keep files from being included when computing dependencies? (The goal here is preventing files from being included in the app bundle.) |
I thought (perhaps erroneously) that those were one and the same. If not, then please disregard :-) |
I think there would be some usecase where you want to use renv for a project, and with some files to publish and some other you want ignore from being bundle, but they be part of your renv project. For rsconnect, today this is only possible to select file to bundle with a file manifest only for file to include. You can't provide excluding pattern. I really believe that providing a mechanism to specify which file to bundle (include and exclude) would be really useful - for such framework as golem, it would be for sure. I think depending on renv for this kind of feature would not be so simple to understand for users. And what if you are using renv also yourself for your project? There would be a double usage: by the user and by rsconnect as a tool to create the bundle. could there be conflict ? Otherwise, would it be possible (and a good idea?) to just take the mechanism you put in renv to be of use in rsconnect ? I am thinking about |
I think Jonathan's take is accurate. The goal in this case is to specify the bundle more explicitly instead declaring its contents in the We would need to be careful here, as any updates to rsconnect would also need to be made to the Git deployments supported by RSC (which does not use reconnect for the bundling step) and perhaps even the Jupyter deployment scenario (for consistency). All that said, this is a relatively common request and I think it'd be a very handy feature! cc: @aronatkins |
Hey, Any update on this issue? I've noticed that this behaviour is possible if you do it "manually", i.e. when you click on the RStudio "blue button", and uncheck files, this will create an entry in the
Then if I call Am I correct in understanding that ignoring files is only available if one does the deployment through point and click? Happy to try and implement this, would you review a PR that does that? |
… of the specified directory
… of the specified directory
I just opened a pull request for including this feature |
It seems like this has been added in #502 (thanks @andyquinterom!) and it now merged and available last version. There is no wildcard support but that is like the manifest file in a way and it is a good start that this feature is available now.
@slopp I don't know if you concerns here were checks. Probably. I think this could be closed now, right ? |
Would be great to also document this in the RSC admin guide :) |
I've been trying to finds docs on the
For instance, the docs don't state whether the Lastly, it appears that Workbench (2022.07.2) doesn't show the |
For what I can tell from the code:
Wildcards will not work (e.g., Are there any plans on updating this feature? Explicitly stating all files/directories that should be included/excluded in a deployment is hard to sustainably maintain. |
Improved documentation will be part of #684. Otherwise, we don't have current plans to enhance this functionality. @nick-youngblut you might be better of generating |
Thanks @hadley for updating the documentation. So I either have to list all files/directories to include or all files/directories to ignore. I'm surprised that Connect content developers are generally OK with this functionality, given that everyone is used to the high functionality of .gitignore (and similar), and the burden of maintaining a full list of file/directories to include or exclude from publishing on Connect. I would think that many users would want to just symlink their From the code shown above ( |
@nick-youngblut if you want to implement it, a PR would be awesome! But I think you should wait until after #684 has been merged, since that pulls out the bundling code into something easier to modify. |
Thank you all for the discussion! Adding that this would also be super helpful for deploying git-backed content. It seems that manually defining appFiles in |
It is already possible to provide a
appFileManifest
indeployApp
to list the files that should be bundle.I am often in the case like with
.gitignore
or.Rbuildignore
: it is easier to tell what we don't want at all instead of listing everything we want.I would find it interesting to have a feature like that, a
.rscignore
file that would allow to always exclude some files to be bundled byrsconnect
.This is like
.Rbuildignore
that prevent bundling in R CMD build.We encountered this need in some scenarios where
Publishing unneeded file is not a problem, but can cause issues because they are scanned for dependencies and can increase the bundle size.
What do you think ?
In a way this issue relates to #342 about how to manage bundle files.
The text was updated successfully, but these errors were encountered: