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

Use Excelx::Shared class to pass shared data to Excelx Sheets. #220

Merged
merged 2 commits into from
Aug 20, 2015

Conversation

stevendaniels
Copy link
Contributor

This is part of a larger set of changes I will be introducing to Roo. These changes will address type-casting issues and memory issues like #179. The goal is for these changes to be included in version 2.1.0 of Roo.

This commit has:

  • added the Excelx::Shared class
  • replaced several instance variables in the Excelx class with Excelx::Shared.
  • modified the arguments for Excelx::Sheet and Excelx::SheetDoc

The next phase will include major changes to Excelx::SheetDoc and Excelx::Cell.

This changes the initializers for Roo::Excelx::Sheet and
Roo::Excelx::SheetDoc, so a minor version bump will be required.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 93.99% when pulling 4b41a2e on stevendaniels:use_shared_class into ae94acc on roo-rb:master.

@simonoff
Copy link
Member

simonoff commented Jun 2, 2015

Looks good.

stevendaniels added a commit that referenced this pull request Aug 20, 2015
Use Excelx::Shared class to pass shared data to Excelx Sheets.
@stevendaniels stevendaniels merged commit 9b4e67c into roo-rb:master Aug 20, 2015
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