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.

Sign up for GitHub

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

Jump to bottom

Rewrote CI to Pascal #32

Merged
merged 10 commits into from
Jan 17, 2025
Merged

Rewrote CI to Pascal #32

merged 10 commits into from
Jan 17, 2025

Conversation

theavege
Copy link
Contributor

What do you think about this CI?

@Xor-el
Copy link
Owner

Xor-el commented Jan 13, 2025

Hello @theavege this is brilliant.

  1. can we get it to work on both Windows and Ubuntu?
  2. How is the FPC/Lazarus specified in this new model or is that determined by the latest Lazarus/FPC version in the respective OS Pacakage Manager?

Sidenote, the current build seems to be failing, can you take a look at this please?

@theavege theavege closed this Jan 14, 2025
@theavege
Copy link
Contributor Author

I fixed CI

@theavege theavege reopened this Jan 14, 2025
@theavege
Copy link
Contributor Author

can we get it to work on both Windows and Ubuntu?

Yes

or is that determined by the latest Lazarus/FPC version in the respective OS Pacakage Manager?

Yes

@Xor-el Xor-el merged commit 6778620 into Xor-el:master Jan 17, 2025
2 checks passed
@theavege theavege deleted the upd/ci branch January 17, 2025 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@theavege @Xor-el