Skip to content

Add publiccode.yml validation CI#1384

Merged
ilgrosso merged 1 commit into
apache:masterfrom
bfabio:publiccode
May 31, 2026
Merged

Add publiccode.yml validation CI#1384
ilgrosso merged 1 commit into
apache:masterfrom
bfabio:publiccode

Conversation

@bfabio
Copy link
Copy Markdown
Contributor

@bfabio bfabio commented May 12, 2026

No description provided.

@ilgrosso
Copy link
Copy Markdown
Member

Thank you for this PR @bfabio

If this is your very first contribution to an ASF project, please ensure to submit your ICLA.

Comment thread .github/workflows/publiccodeyml-check.yml
@ilgrosso
Copy link
Copy Markdown
Member

@bfabio thank you

Can you please also merge the latest changes from master? Or rebase on it. Thx

@bfabio
Copy link
Copy Markdown
Contributor Author

bfabio commented May 13, 2026

Thank you for this PR @bfabio

If this is your very first contribution to an ASF project, please ensure to submit your ICLA.

Hey @ilgrosso, isn't this one covered by clause 5, cause it's such a trivial change:

While small contributions to Apache projects are made under clause 5 of the Apache-2.0 license, all project maintainers of, as well as contributors of any large contributions to, any Apache projects must complete, sign, and submit via email an Individual Contributor License Agreement (ICLA).

@ilgrosso
Copy link
Copy Markdown
Member

it's such a trivial change

the typo change is definitely trivial (and was already fixed by a later commit, that's why I was asking above to merge).

The proposal to add new GH workflow is not, especially because it will likely require to ask ASF infra to authorize italia/publiccode-parser-action

@bfabio
Copy link
Copy Markdown
Contributor Author

bfabio commented May 13, 2026

it's such a trivial change

the typo change is definitely trivial (and was already fixed by a later commit, that's why I was asking above to merge).

The proposal to add new GH workflow is not, especially because it will likely require to ask ASF infra to authorize italia/publiccode-parser-action

Sounds fair, on it

Signed-off-by: Fabio Bonelli <fb@fabiobonelli.it>
@ilgrosso ilgrosso changed the title Add publiccode.yml validation CI and fix typo Add publiccode.yml validation CI May 13, 2026
@ilgrosso
Copy link
Copy Markdown
Member

@bfabio the PR looks ok.

Before merging it, however, italia/publiccode-parser-action shall be allowed by following what reported by https://github.com/apache/infrastructure-actions#submitting-an-action

Please, link here the PR there once submitted, thanks.

@ilgrosso
Copy link
Copy Markdown
Member

No updates for two weeks.

@bfabio please let us know if you have intention to move on with this work, otherwise we will close this PR, thanks.

@bfabio
Copy link
Copy Markdown
Contributor Author

bfabio commented May 25, 2026

Hey :)
No worries, I'll take care of the remaining steps. There was already quite a bit of process/work involved on my side for this PR, so I'm just handling things step by step when I have time.

@ilgrosso
Copy link
Copy Markdown
Member

Sounds good, thanks for heads up

@bfabio
Copy link
Copy Markdown
Contributor Author

bfabio commented May 30, 2026

Done! apache/infrastructure-actions#904

@ilgrosso
Copy link
Copy Markdown
Member

Great job @bfabio going to merge this PR right now.

@ilgrosso ilgrosso merged commit 4612152 into apache:master May 31, 2026
30 of 34 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