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

[pt-PT] Added rule ID:SIMPLIFICAR_A_SER_PP_A_INF #9623

Merged
merged 2 commits into from
Nov 14, 2023

Conversation

marcoagpinto
Copy link
Member

Heya,

A brand-new rule for pt-PT.

Follow the discussion here:
#9622

Thanks!

@marcoagpinto
Copy link
Member Author

@susanaboatto @p-goulart

Why did the test fail?

Help!

@p-goulart
Copy link
Collaborator

p-goulart commented Nov 8, 2023

Why did the test fail?

Caused by: org.xml.sax.SAXParseException; lineNumber: 3337; columnNumber: 79; The entity "adverbios_de_intensidade" was referenced, but not declared.

Did you by any chance create a new entity but forget to commit the entities file?

@marcoagpinto
Copy link
Member Author

@p-goulart

Yes, I forgot the entity.

I have just added it.

Sorry… 😛

@marcoagpinto
Copy link
Member Author

Before going to bed, I wrote down on paper: “don't forget to add inflected=yes, verb "haver”" and in the morning I didn't remember to add the entity used by the rule.

😄 😄 😄 😄

@marcoagpinto marcoagpinto merged commit 586a459 into master Nov 14, 2023
1 check passed
@marcoagpinto marcoagpinto deleted the lt_marcoagpinto_20231108_0654 branch November 14, 2023 19:03
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.

2 participants