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.

Sign up for GitHub

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

Jump to bottom

docs: Reshuffling the content a bit #70

Merged
merged 1 commit into from
Jun 21, 2023
Merged

docs: Reshuffling the content a bit #70

merged 1 commit into from
Jun 21, 2023

Conversation

jukkar
Copy link
Collaborator

@jukkar jukkar commented Jun 20, 2023

  • Affinity document is embedded to topology-aware.md file as it is related to topology-aware policy.
  • Shuffled the table of contents so that it is more logical.

marquiz
marquiz approved these changes Jun 21, 2023
View reviewed changes
Copy link
Collaborator

@marquiz marquiz left a comment

Choose a reason for hiding this comment

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

Thanks @jukkar. Let's get this in! LGTM

I have some further suggestions for improvement but it's easier to submit those as a separate PR on top of this.

@marquiz
Copy link
Collaborator

marquiz commented Jun 21, 2023

ping @klihub

@jukkar jukkar marked this pull request as ready for review June 21, 2023 08:30
@jukkar
docs: Reshuffling the content a bit
195021d 
* Affinity document is embedded to topology-aware.md file
  as it is related to topology-aware policy.
* Shuffled the table of contents so that it is more logical.

Signed-off-by: Jukka Rissanen <[email protected]>
@jukkar
Copy link
Collaborator Author

jukkar commented Jun 21, 2023

Fixed merge conflict.

@klihub klihub merged commit 1452e84 into containers:main Jun 21, 2023
2 checks passed
@jukkar jukkar deleted the fixes/docs branch June 21, 2023 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@klihub klihub klihub approved these changes

@marquiz marquiz marquiz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jukkar @marquiz @klihub