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

Added better CI #23

Merged
merged 1 commit into from
Jun 25, 2024
Merged

Added better CI #23

merged 1 commit into from
Jun 25, 2024

Conversation

orangemug
Copy link
Contributor

@orangemug orangemug commented Jun 25, 2024

This PR adds better CI, dev scripts and testing of published output.

There is a test run over https://github.com/orangemug/OOXML-Validator/actions/runs/9669239324 to view, results shown below

Screenshot 2024-06-25 at 21 37 06

The publish-test-* scripts, run the script and test for the default output "Value cannot be null." on the target environment.

There are also now some development scripts, which get used in CI also (and by me when developing this)

Development

You can run some development scripts with

./dev.sh help
# ./dev.sh <command> [build_env]
# 
#   help                 show this help message
#   docker               docker container for development
#   build <build_env>    build
#   envs                 show build envs
#   run <build_env>      run the command line
# 

To build run the following, replacing linux-x64 with your environment

./dev.sh build linux-x64

Run the executable with

./dev.sh run linux-x64 ./test.docx

If you don't want to install dotnet just run the following to start a container shell

./dev.sh docker
# root@8b0f1aad055c:/code# [RUN YOUR COMMANDS HERE]

@orangemug orangemug marked this pull request as ready for review June 25, 2024 20:57
@mikeebowen
Copy link
Owner

@orangemug This is great, thanks for the contribution!

@mikeebowen mikeebowen merged commit d21c01c into mikeebowen:main Jun 25, 2024
14 checks passed
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