Skip to content

Conversation

@thibaudteil
Copy link
Collaborator

  • Tickets addressed: 1965
  • Review: By commit
  • Merge strategy: Merge (no squash)

Description

Update requirements, link readme file to the wiki which now contains all the build instructions

Verification

None

Documentation

Wiki

Future work

None

@thibaudteil thibaudteil force-pushed the feature/1965-requirements branch 2 times, most recently from d0de77d to 70d9767 Compare December 12, 2025 23:41
Copy link
Collaborator

@owal7428 owal7428 left a comment

Choose a reason for hiding this comment

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

I don't think we need to nuke our README just because we now have Wiki pages. I think we should just streamline it down to just like the overview stuff. I think the Protobuf repo is a good example of this

@thibaudteil thibaudteil force-pushed the feature/1965-requirements branch from 70d9767 to e37c47a Compare December 14, 2025 22:05
@thibaudteil
Copy link
Collaborator Author

I don't think we need to nuke our README just because we now have Wiki pages. I think we should just streamline it down to just like the overview stuff. I think the Protobuf repo is a good example of this

I added a bit more in there. I'm trying to minimize having several sources of truth, but this should be better now

@thibaudteil thibaudteil requested a review from owal7428 December 14, 2025 22:14
@thibaudteil thibaudteil force-pushed the feature/1965-requirements branch from e37c47a to 84925e4 Compare December 15, 2025 20:10
@thibaudteil thibaudteil merged commit 19dc206 into develop Dec 16, 2025
1 check passed
@thibaudteil thibaudteil deleted the feature/1965-requirements branch December 16, 2025 23:56
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.

4 participants