Simplify build configuration and add CI/CD documentation #2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Owner
s97712
commented
Aug 23, 2025
s97712
commented
- Set conditional default version to 1.0.0-dev
- Streamline package release notes
- Remove debug version suffix configuration
- Add automated build and release workflow documentation
- Document manual build process with native library steps
- Set conditional default version to 1.0.0-dev - Streamline package release notes - Remove debug version suffix configuration - Add automated build and release workflow documentation - Document manual build process with native library steps
- Add file listing to verify native library copying - Fix upload artifact path to include native/ subdirectory
Add debugging steps to inspect runtime files structure and test output directory before running tests to help diagnose CI issues.
Remove '/native/' suffix from artifact path to correctly upload the entire runtime directory structure.
Show the directory path before listing contents to improve CI debugging
Remove matrix.rid from artifact path to upload entire runtimes directory
- Merge build-and-test and publish jobs into pack-and-publish - Move .NET build and test steps to build-native job for better parallelization - Add conditional publishing based on tag presence - Remove redundant debug steps from pack-and-publish job
Add LIB_PREFIX variable to handle different library naming conventions between Windows (no prefix) and Unix-like systems (lib prefix).
Add package-test job that runs after pack-and-publish to test the published NuGet package on Linux, macOS, and Windows. The job downloads the published package artifacts, updates the test project to use the specific version, and runs tests to ensure package functionality across all platforms. Also updates default version to 0.0.1-dev for initial development builds.
Move NuGet publishing from pack-and-publish job to separate publish job that runs after package-test job completes. This ensures packages are thoroughly tested across all platforms before being published to NuGet.
The dotnet remove and add commands now specify the full project file path instead of changing directories, making the commands more explicit and avoiding potential path-related issues in the CI environment.
Change the dotnet package source reference in CI workflow to use the more explicit './artifacts' path instead of 'local'.
This ensures dependencies are properly restored before running tests on the TypeScriptParser.TestPackage, preventing potential build failures due to missing dependencies.
Move dotnet restore step to execute before package reference update to ensure dependencies are available when updating package references in the CI workflow.
Remove architecture-specific detection for macOS and default to ARM64, consolidating the two conditional statements into one.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.