Skip to content
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

gemspec: Use Ruby to pick the files to keep #297

Closed
wants to merge 1 commit into from

Conversation

olleolleolle
Copy link
Contributor

This also excludes the gemspec from the files to ship.

This also excludes the gemspec from the files to ship.
@codecov
Copy link

codecov bot commented Apr 13, 2021

Codecov Report

Merging #297 (8df1cbb) into master (45e8769) will not change coverage.
The diff coverage is n/a.

❗ Current head 8df1cbb differs from pull request most recent head 121f236. Consider uploading reports for the commit 121f236 to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##            master      #297   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           50        50           
  Lines         1868      1868           
=========================================
  Hits          1868      1868           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 45e8769...121f236. Read the comment docs.

@ddnexus
Copy link
Owner

ddnexus commented Apr 14, 2021

I prefer the code in this PR, but it will pull in also temporary files or excluded files that happen to be in the lib dir at the moment of the build. The git command would avoid that.

@olleolleolle
Copy link
Contributor Author

olleolleolle commented Apr 14, 2021

I see your point.

One way to get there would be a fully explicit file list, or to work with Rake clean/clobber to point out any "known temporary artifacts", and let a rake build task depend on such a cleanup.

Or, one can use the super-explicit rake-manifest gem, which keeps a manifest file and generates this kind of content.

Alright, now I've typed the alternative ways that I know of.

Ah, yes, one more thing. A reason to do it: There's a rubocop-packaging plugin, with rules about how to make the life of a Debian package maintainer easier - they do repackaging of gems, and any shelling out of Ruby makes their work harder. The plugin has documentation which tries to explain their perspective.

Close or keep this PR, it's your repository. Looks cool!

@ddnexus
Copy link
Owner

ddnexus commented Apr 14, 2021

Thank you for all the suggestions and the motivation of getting this done. I like the rake-manifest gem! That would be the solution to all the problems, and should not get in the way of adding/renaming files. It will explicitly fail, unless you remembered to generate the new manifest, which is a good thing IMO.

If you have time to do it, I would be very happy to merge your PR!
Thank you!

@ddnexus ddnexus changed the base branch from master to dev April 15, 2021 02:40
@ddnexus
Copy link
Owner

ddnexus commented Apr 16, 2021

Implemented! Thank you for the suggestions.

@ddnexus ddnexus closed this Apr 16, 2021
@olleolleolle olleolleolle deleted the patch-2 branch April 16, 2021 07:15
@olleolleolle
Copy link
Contributor Author

@ddnexus Best-possible outcome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants