Skip to content

Split build-test-publish job #30

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

Merged
merged 3 commits into from
Jun 15, 2025
Merged

Conversation

fdcastel
Copy link
Member

@fdcastel fdcastel commented Jun 2, 2025

Proposed fix for #29.

@fdcastel
Copy link
Member Author

fdcastel commented Jun 3, 2025

As mentioned in #29, we should separate the "build-test" step from the "publish" step.

A good example of this can be seen in this recently merged PR:

  • It passed in the official repository.
  • It failed in the fork.

Forks should be able to build and run the tests for the project, but never attempt to publish it.

@fdcastel fdcastel changed the title Stricter condition for build-test-publish job Split build-test-publish job Jun 3, 2025
@fdcastel
Copy link
Member Author

fdcastel commented Jun 3, 2025

New fork-friendly workflows.

CI Workflow

  • Triggered on all pushes and pull requests;
  • Executes on all repositories (including forks);
  • Performs build and test tasks.

Publish Workflow

  • Triggered only by pushes to the master branch of the official repository and once weekly;
  • Runs exclusively on the official repository;
  • Executes build, test, and publish tasks.

All workflows support manual triggering via dispatch.

@fdcastel
Copy link
Member Author

fdcastel commented Jun 3, 2025

Rebased with latest master.

@fdcastel
Copy link
Member Author

Rebased with the latest master.

@fdcastel fdcastel self-assigned this Jun 13, 2025
@fdcastel fdcastel merged commit 7592182 into FirebirdSQL:master Jun 15, 2025
1 check passed
@fdcastel fdcastel deleted the issue-29 branch June 15, 2025 01:54
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