Skip to content

[CALCITE-7460] Broken Maven Central and CI Status badges in README#4862

Draft
caicancai wants to merge 1 commit intoapache:mainfrom
caicancai:7460
Draft

[CALCITE-7460] Broken Maven Central and CI Status badges in README#4862
caicancai wants to merge 1 commit intoapache:mainfrom
caicancai:7460

Conversation

@caicancai
Copy link
Copy Markdown
Member

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 3, 2026

@caicancai
Copy link
Copy Markdown
Member Author

@zabetak I'm not sure if this complies with the guidelines. Could you please take a look if you have time?

Copy link
Copy Markdown
Member

@zabetak zabetak left a comment

Choose a reason for hiding this comment

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

I think its not OK to link to external services for privacy reasons (https://privacy.apache.org/policies/website-policy.html). Based on apache/privacy-website#54 its not OK to use shields.io. If you want a definite answer please send an email to privacy@apache.org

I assume that using maven-badges-herokuapp.com is not OK either.

@caicancai
Copy link
Copy Markdown
Member Author

I think its not OK to link to external services for privacy reasons (https://privacy.apache.org/policies/website-policy.html). Based on apache/privacy-website#54 its not OK to use shields.io. If you want a definite answer please send an email to privacy@apache.org

I assume that using maven-badges-herokuapp.com is not OK either.

Thank you for your review. I will only keep the changes to the CI status and will leave the Maven-related link issues unchanged for now.

@zabetak
Copy link
Copy Markdown
Member

zabetak commented Apr 3, 2026

@caicancai While you are on the topic It would be great if you can follow-up with privacy@apache.org to see if the link to heroku is OK. I feel that this needs to be removed as well.

@caicancai
Copy link
Copy Markdown
Member Author

@caicancai While you are on the topic It would be great if you can follow-up with privacy@apache.org to see if the link to heroku is OK. I feel that this needs to be removed as well.

Okay, I'll send an email to confirm in the next couple of days. Thank you for your suggestion.

@caicancai caicancai marked this pull request as draft April 3, 2026 14:34
@caicancai
Copy link
Copy Markdown
Member Author

I have sent the email and am waiting for a reply.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@calcite.apache.org list. Thank you for your contributions.

@github-actions github-actions Bot added the stale label May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants