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

corrected paths, created directory and moved test files in the directory #140

Merged
merged 3 commits into from
Jul 12, 2019
Merged

corrected paths, created directory and moved test files in the directory #140

merged 3 commits into from
Jul 12, 2019

Conversation

AndrewSerra
Copy link
Contributor

@AndrewSerra AndrewSerra commented Jul 11, 2019

Created the tests directory, moved test files, corrected paths for issue #137 and discussion at issue #132

@atherdon
Copy link
Contributor

we'll wait with this PR. when Shkhiram publish his PR

@AndrewSerra
Copy link
Contributor Author

in the mean time should I start resolving travis-ci errors? @atherdon or move on with tests?

@atherdon
Copy link
Contributor

atherdon commented Jul 12, 2019 via email

@AndrewSerra
Copy link
Contributor Author

@atherdon There is also this to PR merge

@atherdon
Copy link
Contributor

atherdon commented Jul 12, 2019 via email

@atherdon
Copy link
Contributor

i dont think that you made sync correctly. looks like you override changes made by @svr8 with your outdated code

@atherdon atherdon self-requested a review July 12, 2019 18:53
Copy link
Contributor

@atherdon atherdon left a comment

Choose a reason for hiding this comment

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

not cool

@codeclimate
Copy link

codeclimate bot commented Jul 12, 2019

Code Climate has analyzed commit 80a58fe and detected 0 issues on this pull request.

View more on Code Climate.

@atherdon atherdon self-requested a review July 12, 2019 22:19
Copy link
Contributor

@atherdon atherdon left a comment

Choose a reason for hiding this comment

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

i hope you made it by another syncing, not just by copy-pasting that deleted code. or this problem can appear lately

@atherdon atherdon merged commit 4e794c7 into Food-Static-Data:master Jul 12, 2019
@AndrewSerra AndrewSerra deleted the feature/issue137 branch July 15, 2019 15:46
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