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

Typo in osi3::TrafficAction::EndActionsAction documentation #773

Closed
ClemensLinnhoff opened this issue Feb 12, 2024 · 5 comments · Fixed by #774
Closed

Typo in osi3::TrafficAction::EndActionsAction documentation #773

ClemensLinnhoff opened this issue Feb 12, 2024 · 5 comments · Fixed by #774
Assignees
Labels
Documentation Everything which impacts the quality of the documentation and guidelines.

Comments

@ClemensLinnhoff
Copy link
Contributor

Describe the bug

There is a typo in the detailed documentation of osi3::TrafficAction::EndActionsAction: "exection" instead of "execution".

@ClemensLinnhoff ClemensLinnhoff added the Documentation Everything which impacts the quality of the documentation and guidelines. label Feb 12, 2024
@philipwindecker
Copy link
Contributor

I am not employed by this project at the moment. Please assign the issue to someone from the currrent group instead.

@ClemensLinnhoff
Copy link
Contributor Author

@philipwindecker I was not aware of that.

@florianwallerer is there someone else currently responsible for the documentation?

@florianwallerer
Copy link

Hi Clemens,
responsible for the documentation ist the project. (ASAM OSI). As Philip, said, there is no documentation service provider available for ASAM OSI currently.

So probably the best way to fix this is, if you or @PhRosenberger create a pull request for this typo.

@ClemensLinnhoff ClemensLinnhoff linked a pull request Feb 12, 2024 that will close this issue
6 tasks
@ClemensLinnhoff
Copy link
Contributor Author

I quickly checked the other proto files and was a little bit shocked about the number of typos I found in total. Almost half of the files had spelling mistakes. I fixed them in the referenced MR, but we should think about adding a spell check to the CI pipeline.

@philipwindecker
Copy link
Contributor

@philipwindecker I was not aware of that.

@florianwallerer is there someone else currently responsible for the documentation?

No problem. I just wanted to make sure this issue does not sink to the bottom of the Backlog because of this. :)

@pmai pmai closed this as completed in #774 Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Everything which impacts the quality of the documentation and guidelines.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants