Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Build process maintenance #1028

Merged
merged 3 commits into from
Feb 27, 2023
Merged

Build process maintenance #1028

merged 3 commits into from
Feb 27, 2023

Conversation

9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Feb 26, 2023

While trying to understand about the NPOI build process, I found some weird files and build settings in the repository. This PR is small build process maintenance.

  1. Share one assembly sign key file between projects

    There were many same npoi.snk files in the repository.
    Move the key file up to the root of repository, then refer it from each csproj files.

  2. Remove unused assembly sign key file pk.snk

    It's not used anywhere.

  3. Remove unused packages.config files

    Each csproj files are already migrated to use the PackageReference mode.

Move the key file npoi.snk upto the root of repository, then refer it from each csproj files.
Each csproj files are already migrated to use the PackageReference mode.
@tonyqus
Copy link
Member

tonyqus commented Feb 27, 2023

LGTM

@tonyqus tonyqus added this to the NPOI 2.6.1 milestone Feb 27, 2023
@tonyqus tonyqus merged commit 676aca1 into nissl-lab:master Feb 27, 2023
@9rnsr
Copy link
Contributor Author

9rnsr commented Feb 27, 2023

Thanks! I opened #1033 which describes why I started to maintain build process.

@9rnsr 9rnsr deleted the build_maintenance branch February 27, 2023 18:21
@tonyqus tonyqus added the cicd label Jun 22, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants