Skip to content

ci: rewrite our infrastructure#42

Merged
andrepg merged 17 commits into
mainfrom
feat/new-ci-structure
Apr 13, 2026
Merged

ci: rewrite our infrastructure#42
andrepg merged 17 commits into
mainfrom
feat/new-ci-structure

Conversation

@andrepg

@andrepg andrepg commented Apr 13, 2026

Copy link
Copy Markdown
Owner

This PR changes our entire CI structure by splitting some of the files into new workflows. The changes are:

  • Rename bundle-validation.yml ➡️ flatpak-bundle-validation.yml
  • Format tiles to fix Lint and Prettier complains
  • Ignore build and others on Prettier
  • Run i18n-extractor only on PR

Comment thread .github/workflows/i18n-update.yml Outdated
Comment thread .github/workflows/i18n-update.yml Outdated
- fix flatpakref GPG error (use base64 binary export)
- rename pr-checks to quality-checks
- create i18n-update workflow for POT generation
- refactor bundle-validation (split jobs, update triggers)
- remove unnecessary cleanup step from i18n-update
@andrepg

andrepg commented Apr 13, 2026

Copy link
Copy Markdown
Owner Author

Addressed review comments:

  • ✅ Removed unnecessary 'Clean up credentials' step from i18n-update.yml (line 41)

The system dependencies (meson, ninja-build, gjs) are still needed because setup-project action only handles Node.js setup, not the system packages required for the pot-generator script.

@andrepg
andrepg marked this pull request as ready for review April 13, 2026 15:42
@qltysh

qltysh Bot commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

❌ 5 blocking issues (5 total)

Tool Category Rule Count
actionlint Lint specifying action "andrepg/do-it/.github/actions/setup-project" in invalid format because ref is missing. available formats are "{owner}/{repo}@{ref}" or "{owner}/{repo}/{path}@{ref}" 5

Comment thread .github/workflows/flatpak-bundle.yml
Comment thread .github/workflows/i18n-update.yml Outdated

- name: Setup Project
uses: ./.github/actions/setup-project
uses: andrepg/do-it/.github/actions/setup-project

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

specifying action "andrepg/do-it/.github/actions/setup-project" in invalid format because ref is missing. available formats are "{owner}/{repo}@{ref}" or "{owner}/{repo}/{path}@{ref}" [actionlint:action]


- name: Setup Project
uses: ./.github/actions/setup-project
uses: andrepg/do-it/.github/actions/setup-project

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

specifying action "andrepg/do-it/.github/actions/setup-project" in invalid format because ref is missing. available formats are "{owner}/{repo}@{ref}" or "{owner}/{repo}/{path}@{ref}" [actionlint:action]


- name: Setup Project
uses: ./.github/actions/setup-project
uses: andrepg/do-it/.github/actions/setup-project

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

specifying action "andrepg/do-it/.github/actions/setup-project" in invalid format because ref is missing. available formats are "{owner}/{repo}@{ref}" or "{owner}/{repo}/{path}@{ref}" [actionlint:action]

runs-on: ubuntu-latest
steps:
- name: Setup Project
uses: andrepg/do-it/.github/actions/setup-project

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

specifying action "andrepg/do-it/.github/actions/setup-project" in invalid format because ref is missing. available formats are "{owner}/{repo}@{ref}" or "{owner}/{repo}/{path}@{ref}" [actionlint:action]


- name: Setup Project
uses: ./.github/actions/setup-project
uses: andrepg/do-it/.github/actions/setup-project

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

specifying action "andrepg/do-it/.github/actions/setup-project" in invalid format because ref is missing. available formats are "{owner}/{repo}@{ref}" or "{owner}/{repo}/{path}@{ref}" [actionlint:action]

@andrepg
andrepg merged commit 6777529 into main Apr 13, 2026
1 of 2 checks passed
@andrepg
andrepg deleted the feat/new-ci-structure branch April 13, 2026 17:29
@github-project-automation github-project-automation Bot moved this from Planning to Done in ✅️ Do It ! Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant