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.

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

Code Quality: Migrate files from packaging project to main Files.App project #16654

Open
Lamparter opened this issue Dec 24, 2024 · 0 comments
Open

Comments

@Lamparter
Copy link
Contributor

Description

What about starting from merging Files (Package) folder contents into Files.App folder?
Since folder name and project name doesn't match and that way enables us to remove WAP easily later.

This proposal is to move files from the packaging project (Files.Package) to the main project (Files.App), including:

  • Package.appxmanifest
  • Assets/AppTiles/*

While the packaging project can't be fully removed, it would be excellent if these files were moved into the main project and then referenced by the packaging project, like this:

<!-- Inside Files.Package.wapproj -->
<ItemGroup>
  <Content Include="$(RootDirectory)\src\Files.App\Package.appxmanifest">
    <CopyToOutputDirectory>Always</CopyToOutputDirectory>
  </Content>
</ItemGroup>

Concerned code

Contents of Files.Package:

  • Package.appxmanifest
  • Assets/AppTiles/*

Gains

  • Easier overview of package files
  • Allows for gradual removal of WAP project

Requirements

  • Move files to Files.App
  • Reference back in Files.Package.wapproj

Comments

Succeeds #16350

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🆕 New
Development

No branches or pull requests

1 participant