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 chapter08 dependent #8

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

iankode
Copy link

@iankode iankode commented May 7, 2024

resolves Issue #7

without these standard GH files any PR is polluted by bin/debug/obj etc dross files
@marcinjamro
Copy link
Collaborator

Hello!

Thank you for submitting your changes to the code. It looks like someone has modified the code I included in the book and made changes that make it incorrect. As soon as I received notifications about your pull requests, I immediately contacted the publisher and asked to use the exact same version of the code that I provided. I would like to apologize for this situation, but it was not caused by me. I hope the publisher fixes this as soon as possible.

Thanks again for your help and I hope that you enjoy the book!

Have a nice day!
Marcin

@iankode
Copy link
Author

iankode commented May 7, 2024 via email

@marcinjamro
Copy link
Collaborator

Hello!

Thank you for the reply. I hope the publisher will quickly replace the code with its correct version. I must admit that I was very surprised by what I saw in the repository, because unfortunately it was not what I sent to the publisher. Anyway, thank you very much for drawing attention to the problem, because it was thanks to your feedback that I learned about the problem and was also able to react. This will have a positive impact on other developers who want to use the code, as well. So, thank you once again!

Once the publisher places the correct version in the repository, I will be happy to hear your ideas on how the code could be further improved. Feel free to let me know! Each such feedback is very valuable and thanks to it we can further improve our skills. I would also like to get to know you, so if you want, please add me as your friend on LinkedIn.

Have a nice day!
Marcin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants