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 some thoughts on the main README. #78

Merged
merged 2 commits into from
Dec 4, 2024

Conversation

KriSun95
Copy link
Collaborator

@KriSun95 KriSun95 commented Dec 3, 2024

Some additions to the README from our discussion. I have either taken a go at making the change myself or at least edited the lines to leave more of a comment on where I was confused. Feel free to delete the latter edits and replace with your own ones that actually add to the documentation.

@thanasipantazides
Copy link
Contributor

@KriSun95 I just updated the README based on your comments. Very helpful, thank you!

Changes:

  1. There's an test (bin/test_buffers) you can run just to validate the build. Instructions for running and expected output are now in the README.
  2. Wording changes based on your suggestions, (hopefully) clearer overview of systemctl usage, and added debugging info for the Address already in use error and the fstab problems.
  3. Hopefully fixed issue Building the docs #79. Please let me know if building docs works for you now.

@thanasipantazides thanasipantazides merged commit da8fc73 into foxsi:main Dec 4, 2024
1 check passed
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