Skip to content

Conversation

windxu88
Copy link
Contributor

@windxu88 windxu88 commented Jan 22, 2025

User description

Purpose of this PR

I forgot to add BranchNamingPattern and PvpProfilesToCheck settings to Wrench in my previous PR #739.
Add them in this PR and re-generate Wrench jobs.

Ticket/Jira #:
FBX-548 CI: Migrate release process to Wrench/PackageWorks

Testing

Functional Testing status:

Performance Testing status:

Documentation & UX Writing

User facing text to review Details
User interface
Public API docs
Changelog
Documentation halo effect Jira link
User manual
Other docs

Additional information

Note to reviewers:

Reminder:


PR Type

Enhancement


Description

  • Configures Wrench branch naming pattern.

  • Sets Wrench PVP profiles to check.

  • Integrates settings into fbxexporterSettings.cs.

  • Completes Wrench release process setup.


File Walkthrough

Relevant files
Configuration changes
fbxexporterSettings.cs
Configure Wrench release process settings                               

Tools/CI/Settings/fbxexporterSettings.cs

  • Sets the BranchNamingPattern property of the Wrench settings to
    BranchPatterns.ReleaseSlash.
  • Initializes the PvpProfilesToCheck property of the Wrench settings
    with a new HashSet containing "rme" and "supported" profiles.
  • These configurations are applied within the constructor of the
    fbxexporterSettings class.
+4/-1     

@windxu88 windxu88 requested a review from mfe January 22, 2025 17:57
@windxu88
Copy link
Contributor Author

windxu88 commented Sep 3, 2025

/review

@windxu88
Copy link
Contributor Author

windxu88 commented Sep 3, 2025

/describe

@windxu88 windxu88 changed the title Add BranchNamingPattern and PvpProfilesToCheck for Wrench [Test Purpose] Add BranchNamingPattern and PvpProfilesToCheck for Wrench Sep 4, 2025
@windxu88 windxu88 marked this pull request as draft September 4, 2025 15:26
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