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

Fix ./Build disttest ( issue#50 ). #51

Closed
wants to merge 1 commit into from

Conversation

shlomif
Copy link
Contributor

@shlomif shlomif commented Feb 6, 2019

Add missing files to the manifest by running " ./Build manifest ".

Add missing files to the manifest by running " ./Build manifest ".
@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.872% when pulling 2a7fd31 on shlomif:fix-disttest into 55466a1 on davorg:master.

6 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.872% when pulling 2a7fd31 on shlomif:fix-disttest into 55466a1 on davorg:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.872% when pulling 2a7fd31 on shlomif:fix-disttest into 55466a1 on davorg:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.872% when pulling 2a7fd31 on shlomif:fix-disttest into 55466a1 on davorg:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.872% when pulling 2a7fd31 on shlomif:fix-disttest into 55466a1 on davorg:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.872% when pulling 2a7fd31 on shlomif:fix-disttest into 55466a1 on davorg:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.872% when pulling 2a7fd31 on shlomif:fix-disttest into 55466a1 on davorg:master.

@shlomif
Copy link
Contributor Author

shlomif commented Feb 6, 2019

Seems like travis-ci should also test "./Build disttest" but it doesn;t now.

@@ -1,5 +1,5 @@
.travis.yml
Copy link
Collaborator

Choose a reason for hiding this comment

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

this file should be skipped from the Manifest, please add it to MANIFEST.SKIP

@atoomic
Copy link
Collaborator

atoomic commented Feb 6, 2019

Please view comment and remove .travis.yaml by adding it to MANIFEST.SKIP
otherwise this looks good

@davorg
Copy link
Collaborator

davorg commented Feb 6, 2019

Thanks. But I made this fix literally moments before seeing this PR.

@davorg davorg closed this Feb 6, 2019
@shlomif
Copy link
Contributor Author

shlomif commented Feb 6, 2019

@davorg : thanks!

shlomif added a commit to shlomif/xml-feed that referenced this pull request Feb 7, 2019
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.

4 participants