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

Update scenario.md #475

Merged
merged 3 commits into from
Aug 30, 2023
Merged

Update scenario.md #475

merged 3 commits into from
Aug 30, 2023

Conversation

RymMichaut
Copy link
Member

Try to make the description a bit clearer and simpler

Try to make the description a bit clearer and simpler
@FlorianJacta
Copy link
Member

What if we put the use cases before explaining a scenario? Having these use cases makes us understand what they are directly

Copy link
Member

@jrobinAV jrobinAV left a comment

Choose a reason for hiding this comment

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

I believe it s much better. I have a few suggestions though. What do you think?

docs/manuals/core/concepts/scenario.md Outdated Show resolved Hide resolved
docs/manuals/core/concepts/scenario.md Outdated Show resolved Hide resolved
docs/manuals/core/concepts/scenario.md Outdated Show resolved Hide resolved
@jrobinAV jrobinAV self-assigned this Jun 23, 2023
Copy link
Member

@FabienLelaquais FabienLelaquais left a comment

Choose a reason for hiding this comment

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

Minor changes.

Make sure you don't have lines longer than 100 characters: makes editing far easier. Thx

docs/manuals/core/concepts/scenario.md Outdated Show resolved Hide resolved
docs/manuals/core/concepts/scenario.md Outdated Show resolved Hide resolved
docs/manuals/core/concepts/scenario.md Outdated Show resolved Hide resolved
docs/manuals/core/concepts/scenario.md Outdated Show resolved Hide resolved
docs/manuals/core/concepts/scenario.md Outdated Show resolved Hide resolved
docs/manuals/core/concepts/scenario.md Outdated Show resolved Hide resolved
@jrobinAV jrobinAV merged commit 66e9892 into develop Aug 30, 2023
1 check passed
@jrobinAV jrobinAV deleted the RymMichaut-rewrite branch August 30, 2023 12:36
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.

None yet

4 participants