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

Account for new .txt, .md test outputs #327

Merged
merged 2 commits into from
Oct 15, 2015
Merged

Account for new .txt, .md test outputs #327

merged 2 commits into from
Oct 15, 2015

Conversation

hexylena
Copy link
Member

No description provided.

@hexylena hexylena added the bug label Oct 15, 2015
@peterjc
Copy link
Contributor

peterjc commented Oct 15, 2015

Does it make sense to add these to .gitignore as well?

@hexylena
Copy link
Member Author

@peterjc yep, they shouldn't be included in your repository. Or did you mean a gitignore as part of planemo that I'm not aware of?

@nsoranzo
Copy link
Member

I guess Peter means https://github.com/galaxyproject/planemo/blob/master/.gitignore , which I'd +1.

@hexylena
Copy link
Member Author

Done

@martenson
Copy link
Member

👍

@peterjc
Copy link
Contributor

peterjc commented Oct 15, 2015

Yes, that's what I had in mind with .gitignore - I do sometimes run planemo from the repository using relative or absolute paths to my tool repositories.

+1 on merging assuming the tests come back fine.

jmchilton added a commit that referenced this pull request Oct 15, 2015
Account for new .txt, .md test outputs
@jmchilton jmchilton merged commit 6c0dc20 into master Oct 15, 2015
@jmchilton jmchilton deleted the missing-filters branch October 15, 2015 17:42
@jmchilton jmchilton mentioned this pull request Oct 15, 2015
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants