Skip to content

Added some books and resources#91

Merged
glyn merged 3 commits into
lborb:mainfrom
nzec:patch-1
Dec 8, 2025
Merged

Added some books and resources#91
glyn merged 3 commits into
lborb:mainfrom
nzec:patch-1

Conversation

@nzec

@nzec nzec commented Oct 24, 2025

Copy link
Copy Markdown
Contributor
  • Hypervisor, OS tutorial
  • Ferrous Systems books
  • Atomics and Locks
  • Rust for Rustaceans

Please feel free to squash the commits

@glyn glyn left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread src/unofficial.md Outdated
Comment thread src/unofficial.md Outdated
@nzec

nzec commented Nov 1, 2025

Copy link
Copy Markdown
Contributor Author

@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.

@glyn glyn left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread src/applications.md Outdated
Comment thread src/applications.md Outdated
Comment thread src/unofficial.md Outdated
@nzec

nzec commented Nov 5, 2025

Copy link
Copy Markdown
Contributor Author

Did the changes @glyn

@glyn glyn left a comment

Copy link
Copy Markdown
Contributor

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?

Comment thread src/applications.md Outdated
Comment thread src/applications.md

@glyn glyn left a comment

Copy link
Copy Markdown
Contributor

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

nzec commented Dec 10, 2025

Copy link
Copy Markdown
Contributor Author

@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

glyn commented Dec 10, 2025

Copy link
Copy Markdown
Contributor

@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