Skip to content
This repository was archived by the owner on Dec 5, 2021. It is now read-only.

Some generic exceptions refactored into custom and specific exceptions#410

Open
mykolasglinskis wants to merge 4 commits into
sochix:masterfrom
mykolasglinskis:master
Open

Some generic exceptions refactored into custom and specific exceptions#410
mykolasglinskis wants to merge 4 commits into
sochix:masterfrom
mykolasglinskis:master

Conversation

@mykolasglinskis

Copy link
Copy Markdown
Contributor

No description provided.

@knocte

knocte commented Feb 25, 2017

Copy link
Copy Markdown
Collaborator

Thanks for your contribution! You don't need the $ character of strings because you're not using string interpolation in them. If you remove that I'll merge this patch.

@knocte

knocte commented Feb 25, 2017

Copy link
Copy Markdown
Collaborator

(PS: You don't need to close this PR and open a new one to be able to make changes.)

@mykolasglinskis

Copy link
Copy Markdown
Contributor Author

removed interpolation

@knocte

knocte commented Mar 29, 2017

Copy link
Copy Markdown
Collaborator

Your last commit is introducing unnecessary diff noise, did you change the line endings or something?

@knocte

knocte commented Apr 24, 2017

Copy link
Copy Markdown
Collaborator

hello?

@knocte

knocte commented Sep 18, 2017

Copy link
Copy Markdown
Collaborator

@mykolasglinskis ping?

@dev-masih

dev-masih commented Mar 21, 2018

Copy link
Copy Markdown
Contributor

@knocte so the original poster wont answer anymore so i propose to close this PR and i Open another with the same changes so PR can merge into project. what do you think?

@knocte

knocte commented Mar 21, 2018

Copy link
Copy Markdown
Collaborator

@masihakbari yeah sure, we can close this one when you have opened a new one.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants