Skip to content

ci: refactor: update tests and format CIs to handle cross-platform#28

Merged
Abega1642 merged 5 commits intomainfrom
ci/ci-format--ci-test
Jan 9, 2026
Merged

ci: refactor: update tests and format CIs to handle cross-platform#28
Abega1642 merged 5 commits intomainfrom
ci/ci-format--ci-test

Conversation

@Abega1642
Copy link
Owner

Problem:

  • When we run make format to Unix system, everything goes well. but when it's run on Windows, it breaks because some Windows system does not have the bash interpreter installed.
  • Some tests are sucessful on Unix system but not on WIndows when we run them on real Windows systems.

Solution:

It is clear that we won't force the developer to install all of the required dependencies on their OS just to run the make format or more critical again if they try to generate an ar-infra project using ar-infra-cli.

what we did here :

  • we create a format.bat, a Windows executable to handle Windows systems that does not have bash interpreter directly
  • we update the Makefile to handle both Unix and Windows executable of formatting script.
  • we update the ci-test.yml and ci-format.yml to handle the cross-platform system (Linux, MacOS, Windows)
  • we separate the unit test and integration test.

@Abega1642 Abega1642 self-assigned this Jan 9, 2026
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2026

@Abega1642 Abega1642 merged commit 8a061a2 into main Jan 9, 2026
27 checks passed
@Abega1642 Abega1642 deleted the ci/ci-format--ci-test branch January 16, 2026 12:30
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.

1 participant