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

Remove test files from package #63

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

f440
Copy link
Contributor

@f440 f440 commented Jul 15, 2024

Currently, it has become standard practice not to include test files in Gem file packages. In this pull request, we have ensured that test files are not included in the package by updating the gemspec file.

Changes Made

  • Modified files to exclude test files from the package.
  • Removed the deprecated test_files specification as it is no longer recommended.

Results

  • The file size after extraction has been reduced from 1068 KB to 588 KB

Benefits

  • Shorter download times for CI and developers
  • Reduced container sizes when the gem is included

References

@hefa hefa self-assigned this Jul 19, 2024
@hefa
Copy link
Member

hefa commented Jul 19, 2024

Thank you for the PR. We will review it.

@hidebike712 hidebike712 merged commit 3bac040 into authlete:master Jul 19, 2024
@f440 f440 deleted the remove-unused-files-from-package branch July 19, 2024 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants