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

Added the FluidConsole project #238

Open
wants to merge 9 commits into
base: Devt
Choose a base branch
from
Open

Added the FluidConsole project #238

wants to merge 9 commits into from

Conversation

bberkhin
Copy link

@bberkhin bberkhin commented Jan 9, 2022

Added the FluidConsole project. The project is a win32 console application that allows you to debug and emulate Fluidnc code on Windows. File "config.yaml" should be placed on the working directory. Print "COM#" in command line to connect via com port

Added the FluidConsole project. The project is a win32 console application that allows you to debug and emulate Fluidnc code on Windows. File "config.yaml" should be placed on the working directory. Use COM# in command line to connect via com port
Copy link
Collaborator

@MitchBradley MitchBradley left a comment

Choose a reason for hiding this comment

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

This code looks very interesting but there are some stylistic things that must be addressed before we can let it in the main code base.

FluidNC/src/Main.cpp Outdated Show resolved Hide resolved
FluidNC/src/Protocol.cpp Outdated Show resolved Hide resolved
FluidNC/src/Protocol.cpp Outdated Show resolved Hide resolved
X86TestSupport/TestSupport/ComPortX86.cpp Show resolved Hide resolved
config.yaml Outdated Show resolved Hide resolved
@bberkhin
Copy link
Author

Done.
Have a look, pls

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