Skip to content
This repository has been archived by the owner on Sep 23, 2023. It is now read-only.

Strip whitespace from column headers in mapping file. #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Strip whitespace from column headers in mapping file. #5

wants to merge 1 commit into from

Conversation

kylebittinger
Copy link
Member

Per our discussion, strip whitespace from column names in a mapping file.

@gregcaporaso
Copy link
Contributor

Is someone reviewing this for merge?

@joey711
Copy link
Contributor

joey711 commented Nov 10, 2013

Sorry @kylebittinger, my devel time has been swamped with a backlog of issues for phyloseq. Those are under control now to some extent, so I can try and take a look. I also need to remind myself what we consider necessary for this package before submitting it to CRAN.

@kylebittinger
Copy link
Member Author

I think we're pretty close to an initial release.

An example dataset would be the obvious thing to add. However, one of our
goals is using qiime.io as a back-end package for phyloseq and qiimer. To
meet this goal, I think we only need to shore up the existing tests (if
needed), deal with corner cases correctly, and make sure we have the man
pages in order.

--Kyle

On Sun, Nov 10, 2013 at 5:53 PM, Paul J. McMurdie
notifications@github.comwrote:

Sorry @kylebittinger https://github.com/kylebittinger, my devel time
has been swamped with a backlog of issues for phyloseq. Those are under
control now to some extent, so I can try and take a look. I also need to
remind myself what we consider necessary for this package before submitting
it to CRAN.


Reply to this email directly or view it on GitHubhttps://github.com//pull/5#issuecomment-28163438
.

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

Successfully merging this pull request may close these issues.

3 participants