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

Enhancement: Some publish logs cosmetics #5917

Merged
merged 7 commits into from
Nov 15, 2023
Merged

Enhancement: Some publish logs cosmetics #5917

merged 7 commits into from
Nov 15, 2023
+36 −34

Conversation

BigRoy
Copy link
Collaborator

@BigRoy BigRoy commented Nov 15, 2023

Changelog Description

General logging message tweaks:

  • Sort some lists of folder/filenames so they appear sorted in the logs
  • Fix some grammar / typos
  • In some cases provide slightly more information in a log

Additional info

n/a

Testing notes:

  1. Publish a few things, e.g. a few models and a deadline render
  2. Test a few Ftrack actions - even though, honestly, they are just logging messages tweaks - it shouldn't break

@BigRoy BigRoy added type: refactor Structural changes not affecting functionality community contribution labels Nov 15, 2023
@ynbot ynbot added size/XS Denotes a PR changes 0-99 lines, ignoring general files type: enhancement Enhancements to existing functionality host: TV Paint module: Ftrack labels Nov 15, 2023
@iLLiCiTiT iLLiCiTiT merged commit 91e230a into ynput:develop Nov 15, 2023
8 checks passed
@ynbot ynbot added this to the next-patch milestone Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iLLiCiTiT iLLiCiTiT iLLiCiTiT approved these changes

Assignees
No one assigned
Labels
community contribution host: TV Paint module: Ftrack size/XS Denotes a PR changes 0-99 lines, ignoring general files type: enhancement Enhancements to existing functionality type: refactor Structural changes not affecting functionality
Projects
PR reviewing
Archived in project
Milestone
3.17.6
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@BigRoy @iLLiCiTiT @ynbot