Skip to content
This repository has been archived by the owner on Nov 22, 2024. It is now read-only.

Suggestion on changes for bug #21 #24

Open
wants to merge 3 commits into
base: top_20_drugs
Choose a base branch
from

Conversation

lollanboll
Copy link
Collaborator

Can this be a possible solution?

@lollanboll lollanboll added the enhancement New feature or request label Feb 28, 2024
@villiamriegler villiamriegler linked an issue Feb 28, 2024 that may be closed by this pull request
Copy link
Owner

@villiamriegler villiamriegler left a comment

Choose a reason for hiding this comment

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

I lack domain knowlage for this, approve to not block. Interpret as appoved if @Firverior02 approves as he authoured the changed file.

@lollanboll
Copy link
Collaborator Author

We noticed that this code doesn't fix all cases only most. Will update and fix most likely with Regex later on this week.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When translating spaces after . are removed
2 participants