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

Add Ostinato lab example ost-srl #2111

Merged
merged 3 commits into from
Jun 26, 2024
Merged

Add Ostinato lab example ost-srl #2111

merged 3 commits into from
Jun 26, 2024

Conversation

pstavirs
Copy link
Contributor

@pstavirs pstavirs commented Jun 26, 2024

This PR is work-in-progress awaiting #2110 to be merged first so that the diagram URL can be updated in the lab example before merging this PR

#2110 is merged. This PR is ready for merge now.

Previous diagram URL was pointing to the diagram PR file. Now that the
diagram PR has been merged changed the URL to the actual final URL
@pstavirs pstavirs changed the title [WIP] Add Ostinato lab example ost-srl Add Ostinato lab example ost-srl Jun 26, 2024
@hellt
Copy link
Member

hellt commented Jun 26, 2024

Thanks @pstavirs

Looks good, made tiny edits style-wise, the content looks great. Merging

@hellt hellt merged commit 10b15ba into srl-labs:main Jun 26, 2024
20 checks passed
@pstavirs pstavirs deleted the ost-lab branch June 26, 2024 10:40
@pstavirs
Copy link
Contributor Author

Thanks @hellt for the fast merge. Some bullet list item indentation got removed by mistake - raised a minor #2112

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