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

Some small internal fixes #39

Merged
merged 8 commits into from
Jun 8, 2016
Merged

Some small internal fixes #39

merged 8 commits into from
Jun 8, 2016

Conversation

Scriptodude
Copy link
Collaborator

Looks like #35 didn't do it's whole job.

This PR simply comes and fixes what #35 didn't do.
I have another PR incoming implementing taxes, with some conflicts. It will make it easier to merge this, then do the other PR to remove some modified files.

@ventilooo Ready for merging ^.^

@coveralls
Copy link

coveralls commented Jun 8, 2016

Coverage Status

Coverage remained the same at 52.863% when pulling 60b8f1e on Scriptodude:build into 14c30d1 on AESSUQAM:develop.

@ventilooo ventilooo merged commit 8b4219b into AESSUQAM:develop Jun 8, 2016
@ventilooo
Copy link
Member

Good to me !

@Scriptodude
Copy link
Collaborator Author

@ventilooo Il semblerait que ça n'ait pas encore merger 😩 Ou il y a des fichiers qui sont ignorés, c'est vraiment weird.

Je peux refaire un PR (rien touché du tout de la journée) avec ma branche "build" et pourtant y'a encore des fichiers non-changés dans AESSUQAM:develop 😷

Des idées, car là je comprend que-dalle.

@ventilooo
Copy link
Member

vérifie les commit, tous ceux de ta branche date d'hier et plus encore.

@Scriptodude Scriptodude deleted the build branch June 12, 2016 01:04
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.

None yet

3 participants