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 has_rdoc from gemspec since the method is deprecated #1

Merged
merged 1 commit into from
Sep 27, 2018

Conversation

nickcluc
Copy link

this PR fixes a deprecation raised by Rubygems 1.7.0, also there is a PR for the main axlsx gem that handles this same issue:
randym#593

Once this is pulled in, we should look at updating the Gemfile to use the main version, since @wpeterson's previous issue fix was also merged for the next release.

Copy link

@wpeterson wpeterson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

FYI: I merged my frozen string fix into master of this fork, so once this PR is merged we should have both changes we need and can install from source referencing this repo without a branch (master).

@nickcluc nickcluc merged commit f83e3f3 into master Sep 27, 2018
@nickcluc nickcluc deleted the nc-remove-has-rdoc-gemspec branch September 27, 2018 16:32
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