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

Implement hotreloading for macOS and Windows and other bugfixes #74

Closed
wants to merge 5 commits into from

Conversation

Markos-Th09
Copy link
Contributor

@Markos-Th09 Markos-Th09 commented Jan 26, 2024

Implements hot reloading for macOS and windows

Closes #45
Closes #73

@Markos-Th09 Markos-Th09 changed the title Implement hotreloading for macOS and Windows Implement hotreloading for macOS ~~and Windows~~ Jan 26, 2024
@Markos-Th09 Markos-Th09 changed the title Implement hotreloading for macOS ~~and Windows~~ Implement hotreloading for macOS Jan 26, 2024
@Markos-Th09 Markos-Th09 marked this pull request as ready for review January 26, 2024 15:31
@Markos-Th09 Markos-Th09 changed the title Implement hotreloading for macOS Implement hotreloading for macOS and Windows Jan 26, 2024
@Markos-Th09 Markos-Th09 changed the title Implement hotreloading for macOS and Windows Implement hotreloading for macOS and Windows and other bugfixes Jan 27, 2024
@rexim
Copy link
Member

rexim commented Feb 6, 2024

Thank you for the PR! Unfortunately I do not accept feature implementations as per CONTRIBUTING.md sorry.

@rexim rexim closed this Feb 6, 2024
@Markos-Th09
Copy link
Contributor Author

I understand. Are you planning to implement hotreloading on other platforms yourself any time soon?

@Markos-Th09
Copy link
Contributor Author

I will extract the bug fixes and typo corrections into a new PR without the hotreloading part

@Markos-Th09
Copy link
Contributor Author

Markos-Th09 commented Feb 26, 2024

Why did you deny this PR but allowed #81 to be merged, which is clearly also a feature implementation? Is there something I am not understanding correctly from CONTRIBUTING.md?

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.

I cannot build on Windows ... Build issue: Make readdir, closedir, opendir static in nob.h
2 participants