Skip to content

Conversation

@rantoniuk
Copy link

Copilot AI review requested due to automatic review settings August 12, 2025 16:22
@vercel
Copy link

vercel bot commented Aug 12, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Project Deployment Preview Comments Updated (UTC)
publiccode-yml Ready Preview Comment Aug 12, 2025 4:28pm

This comment was marked as spam.

@rantoniuk rantoniuk changed the title Rename platforms to clientPlatforms and update examples docs: rename platforms to clientPlatforms and update examples Aug 12, 2025
@bfabio
Copy link
Contributor

bfabio commented Aug 12, 2025

Why is this garbage Copilot bot commenting on PRs?

@rantoniuk
Copy link
Author

@https://github.com/apps/copilot-pull-request-reviewer Copilot AI review requested due to automatic review settings

@publiccodeyml publiccodeyml deleted a comment from Copilot AI Aug 13, 2025
@bfabio
Copy link
Contributor

bfabio commented Aug 14, 2025

@rantoniuk thanks for the PR!

FYI, changing the name would be a breaking change, and breaking changes need to go through a process of deprecating the thing first: see https://github.com/publiccodeyml/publiccode.yml/blob/main/governance/procedure-proposing-changes-and-voting.md

Anyway, I don't think changing the name would be necessary, as we can make clear what that key is about by improving its description. The description, on the other hand, badly needs to be improved because right now it is pretty cloudy, and I think it's because the meaning of the key itself is pretty cloudy :)

I'd like to discuss and have a consensus on what this key is supposed to achieve on #221 before

@rantoniuk
Copy link
Author

I'd like to discuss and have a consensus on what this key is supposed to achieve on #221 before

Didn't we already discuss and agree?

@bfabio
Copy link
Contributor

bfabio commented Aug 14, 2025

I'd like to discuss and have a consensus on what this key is supposed to achieve on #221 before

Didn't we already discuss and agree?

I haven't really shared my thoughts yet. I have something else in mind, let me write that down in the issue.

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