Skip to content

Conversation

@nzec
Copy link
Contributor

@nzec nzec commented Oct 24, 2025

  • Hypervisor, OS tutorial
  • Ferrous Systems books
  • Atomics and Locks
  • Rust for Rustaceans

Please feel free to squash the commits

Copy link
Contributor

@glyn glyn left a comment

Choose a reason for hiding this comment

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

Thanks for these suggestions. Please see my review comments.

@nzec
Copy link
Contributor Author

nzec commented Nov 1, 2025

@glyn I have made the changes. I also sorted the unofficial book list alphabetically. I kept that as a separate commit so that the git history is cleaner.

Copy link
Contributor

@glyn glyn left a comment

Choose a reason for hiding this comment

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

Some more italics need adding.

@nzec
Copy link
Contributor Author

nzec commented Nov 5, 2025

Did the changes @glyn

Copy link
Contributor

@glyn glyn left a comment

Choose a reason for hiding this comment

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

Another alphanumeric ordering issue.

Shall we get the current set of changes correct and merged and leave others for a future PR?

Copy link
Contributor

@glyn glyn left a comment

Choose a reason for hiding this comment

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

Thanks for making the changes. For future reference, I would prefer PR scope to be stable after submission. If you find changes, other than response to my review, you'd like to make, please submit a separate PR. This will help me get your PRs merged quickly. The same approach applies in many projects, which is why I point it out. Cheers.

@glyn glyn merged commit 4795b51 into lborb:main Dec 8, 2025
1 check passed
@nzec
Copy link
Contributor Author

nzec commented Dec 10, 2025

@glyn I apologize for the long back and forth. The convention for italics could be mentioned in the README.md itself.

The categorization could also be made better and less confusing. Some of the books in unofficial.md could be moved to applications.md. I'd send a patch if it makes sense.

@glyn
Copy link
Contributor

glyn commented Dec 10, 2025

@glyn I apologize for the long back and forth. The convention for italics could be mentioned in the README.md itself.

Good idea. See #92.

The categorization could also be made better and less confusing. Some of the books in unofficial.md could be moved to applications.md. I'd send a patch if it makes sense.

Ok. If you are unsure, please start a discussion thread or send a draft PR.

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