Skip to content

fix: handle exceptions without a message attribute on compile translations e.g. FileNotFoundError - #1

Merged
ArturGaspar merged 1 commit into
opencraft-release/2.1.0from
artur/compile-translations-error-handling
Dec 25, 2023
Merged

fix: handle exceptions without a message attribute on compile translations e.g. FileNotFoundError#1
ArturGaspar merged 1 commit into
opencraft-release/2.1.0from
artur/compile-translations-error-handling

Conversation

@ArturGaspar

Copy link
Copy Markdown

Description

Backports openedx#74.

Avoids causing another exception in the exception handling of compile_translations() in installation.

Testing instructions

  1. Install in an environment where compiling translations will fail with FileNotFoundError (e.g. if msgfmt is not installed)
  2. See that installation succeeds

Other information

Private-ref: https://tasks.opencraft.com/browse/BB-8077

@viadanna viadanna left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

  • I tested this on a sandbox
  • I read through the code
  • Includes documentation

@ArturGaspar
ArturGaspar merged commit a61c1fa into opencraft-release/2.1.0 Dec 25, 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

Successfully merging this pull request may close these issues.

2 participants