Skip to content

Conversation

@souvikg10
Copy link
Contributor

We have noticed that often in spoken german, people are using zwo as an alternative to zwei. I tried to add in the german NumParser and corresponding rules. let me know if this is okay or i should make some changes to get it working.

Thanks

@souvikg10
Copy link
Contributor Author

@patapizza - hi Julien, is this repository not maintained anymore?

runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [ubuntu-latest, macOS-latest, windows-latest]
Copy link
Contributor

Choose a reason for hiding this comment

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

why delete these?

Dockerfile Outdated
EXPOSE 8000

CMD ["duckling-example-exe", "-p", "8000"]
CMD ["duckling-example-exe", "-p", "8000", "--no-access-log", "--no-error-log"]
Copy link
Contributor

Choose a reason for hiding this comment

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

these should be available by default

import Data.String
import Prelude

newtype Parser a = Parser {runParser :: String -> Maybe (a, String)}
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm pretty sure this parser type + NumParser stuff already exists elsewhere in the code case, you should import it

@souvikg10 souvikg10 reopened this Oct 18, 2023
@souvikg10
Copy link
Contributor Author

Hi @chessai ,
Unfortunately my previous fork got a little messy because it diverged. This time i made sure i only made changes where it is needed.

thank you for your time and attention on this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants