SG-44793 Strip common .tkdeploy exclusions now covered by global defaults

[julien-lang]

Problem

tk-framework-desktopserver was shipping several files that should never reach App Store installs: tests/, .travis.yml, appveyor.yml, and the azure-pipelines/ directory.

Changes

Common CI/dev entries (tests, .travis.yml, appveyor.yml) are now covered by global defaults added to the App Store publish tooling. The .tkdeploy is simplified to only the repo-specific entries: the azure-pipelines/ directory (a folder distinct from azure-pipelines.yml which is handled globally) and the resources/python/ build artefacts.

Note: these changes take effect only once the corresponding upstream tooling change is merged.

Related PRs

• SG-44793 Remove .tkdeploy: all exclusions are now global defaults tk-core#1130 — same cleanup for tk-core
• SG-44793 Strip common .tkdeploy exclusions now covered by global defaults tk-desktop#226 — same cleanup for tk-desktop

Testing

Confirmed all removed entries are covered by the new global defaults. The retained resources/python/* and azure-pipelines entries are specific to this repo.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.82%. Comparing base (5db5ac0) to head (8ce2b11).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #366   +/-   ##
=======================================
  Coverage   65.82%   65.82%           
=======================================
  Files          24       24           
  Lines        1870     1870           
=======================================
  Hits         1231     1231           
  Misses        639      639           

Flag	Coverage Δ
Linux	47.54% <ø> (ø)
Python-3.10	65.82% <ø> (ø)
Python-3.11	65.82% <ø> (ø)
Python-3.13	65.72% <ø> (ø)
Python-3.9	65.79% <ø> (ø)
Windows	63.80% <ø> (ø)
macOS	64.34% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

Simplifies the repository’s .tkdeploy packaging exclusions now that common CI/dev files are handled by global App Store publish defaults, leaving only exclusions that are specific to tk-framework-desktopserver.

Changes:

• Removed repo-local excludes for common CI/dev artifacts (e.g., tests/, .travis.yml, appveyor.yml) now covered by upstream defaults.
• Kept/expression-adjusted repo-specific excludes for azure-pipelines/ and resources/python/ build/pipeline artifacts and helper scripts.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[carlos-villavicencio-adsk]

Same as shotgunsoftware/tk-core#1130, why not entirely removing the .tkdeploy file?

[julien-lang]

Same as shotgunsoftware/tk-core#1130, why not entirely removing the .tkdeploy file?

Same answer as shotgunsoftware/tk-desktop#226 (comment), the resource folder is not excluded by default. And this is a good thing because, here, in this repository, we need to ship some its content.