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

Error with dieresis in expected cfdi #130

Closed
andreagidaltig opened this issue Nov 29, 2023 · 3 comments
Closed

Error with dieresis in expected cfdi #130

andreagidaltig opened this issue Nov 29, 2023 · 3 comments

Comments

@andreagidaltig
Copy link

andreagidaltig commented Nov 29, 2023

I'm working in the module l10n_mx_edi_payslip in v17 and I got an error with dieresis in the expected_cfdi.xml file used in unit tests. It is expected that dieresis are used in this file because it's the structure requested for fiscal uses, so it can't be removed.

This is the error in pipelines:
https://git.vauxoo.com/vauxoo-dev/l10n-mx-payroll/-/jobs/1446371#L51

@andreagidaltig
Copy link
Author

Hi @antonag32 could you help me with this please?

@luisg123v
Copy link
Contributor

Hi @andreagidaltig,

Could you share the relevant part of the pipeline as code, please?

Take into account jobs could be erased at any time.

@luisg123v
Copy link
Contributor

luisg123v commented Nov 29, 2023

AFAIK this is due to prettier/plugin-xml#607, which in turn is caused by SAP/xml-tools#407. If that's the case, there's not too much we could do from our side.

I'd recommend implementing a solution similar to this one, i.e. using a .prettierignore file.

Because of the above, I'm therefore closing this issue as unfixeable. However, feel free to comment if you think I'm mistaking on this.

Regards,

@luisg123v luisg123v closed this as not planned Won't fix, can't repro, duplicate, stale Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants