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

[Repairibility]: fixed false positives of NPEfix on repogit4test data #112

Merged
merged 2 commits into from
Jul 19, 2019

Conversation

niloofartrg
Copy link
Collaborator

No description provided.

@monperrus
Copy link
Contributor

cannot reproduce the failure locally, so we have to fix #115 first and then restart the build here to understand the problem.

@niloofartrg niloofartrg reopened this Jul 19, 2019
@niloofartrg
Copy link
Collaborator Author

I am replacing prints with log but I don't know where to look in the Travis output to see if the problem is fixed, I don't find a max Travis limit error in the CI's output. @monperrus @martinezmatias

@monperrus
Copy link
Contributor

CI is broken because of continuous deployment. Fixed it in c04b486.

@monperrus monperrus merged commit 6a3878e into master Jul 19, 2019
@monperrus
Copy link
Contributor

We're making progress, at least one build passes https://travis-ci.org/SpoonLabs/coming/builds/560932472

Merging to move forward

@niloofartrg
Copy link
Collaborator Author

niloofartrg commented Jul 19, 2019

what you merged has the changed for NPEfix and also log. Is that fine? I was trying to go back in the commits and create a separate PR for logs. @monperrus

@monperrus
Copy link
Contributor

monperrus commented Jul 19, 2019 via email

@monperrus monperrus deleted the NPEfixNew branch October 26, 2019 19:07
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.

2 participants