Skip to content

Fix typo and doc of SharpsDisposal#239

Merged
skinkie merged 2 commits intoTransmodelEcosystem:masterfrom
frodrigo:patch-6
Sep 23, 2021
Merged

Fix typo and doc of SharpsDisposal#239
skinkie merged 2 commits intoTransmodelEcosystem:masterfrom
frodrigo:patch-6

Conversation

@frodrigo
Copy link
Copy Markdown
Contributor

No description provided.

@skinkie skinkie added bug Technical mistake, inconsistency with the documentation, etc. typo Typo (comment or mispelling) correction without any technical impact labels Sep 17, 2021
@skinkie
Copy link
Copy Markdown
Contributor

skinkie commented Sep 17, 2021

Great catch!!

@skinkie
Copy link
Copy Markdown
Contributor

skinkie commented Sep 17, 2021

I would support an XML example with this PR.

@frodrigo
Copy link
Copy Markdown
Contributor Author

I would support an XML example with this PR.

Not sure what you are asking for.

@skinkie
Copy link
Copy Markdown
Contributor

skinkie commented Sep 17, 2021

You are updating the XSD due to a bug. But if we want to make sure that nobody is affected by such changes in the future, we should make an example using this element that is (automatically) validated prior to merging.

Copy link
Copy Markdown
Contributor

@Aurige Aurige left a comment

Choose a reason for hiding this comment

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

some risks with backwarrd compatibility, but obviously a bug/typo to solve

@skinkie
Copy link
Copy Markdown
Contributor

skinkie commented Sep 23, 2021

@Aurige the quicker they are resolved, the less impact ;)

@skinkie skinkie merged commit 915ab9d into TransmodelEcosystem:master Sep 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Technical mistake, inconsistency with the documentation, etc. typo Typo (comment or mispelling) correction without any technical impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants