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.

Sign up for GitHub

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

Jump to bottom

Cylc review++ #81

Draft
wants to merge 3 commits into
base: 7.8.x
Choose a base branch
from
Draft

Cylc review++ #81

wants to merge 3 commits into from

Commits on Oct 30, 2024

  1. review: handle UTF-8 characters in log files 

    * Assume UTF-8.
* Replace any unknown characters to avoid errors.
* Closes cylc#6171
    @oliver-sanders
    oliver-sanders committed Oct 30, 2024
    Configuration menu
    Copy the full SHA
    c4f5f8b View commit details
    Browse the repository at this point in the history

Commits on Oct 31, 2024

  1. review: handle workflow IDs with non-ASCII characters. 

    * Addresses cylc#3441
* Patch a urllib method that does not support non-ASCII characters.
* Search functionality will not work, this is challenging to fix under
  Python 2.
    @oliver-sanders
    oliver-sanders committed Oct 31, 2024
    Configuration menu
    Copy the full SHA
    b708330 View commit details
    Browse the repository at this point in the history

  2. review: restrict symlink scope

    @oliver-sanders
    oliver-sanders committed Oct 31, 2024
    Configuration menu
    Copy the full SHA
    e4da0b6 View commit details
    Browse the repository at this point in the history