Skip to content

ci: secure the license file #402

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

ci: secure the license file #402

wants to merge 2 commits into from

Conversation

tdstein
Copy link
Collaborator

@tdstein tdstein commented Apr 2, 2025

I stumbled across https://github.com/posit-dev/connect-architectures/blob/35626310f6aad0854ea62b151560eaff5a7f57a9/.github/workflows/ci.yml\#L91C1-L96C32 and noticed the license file management is more secure. Also, I found that I prefer the style used.

Copy link

github-actions bot commented Apr 2, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1880 1771 94% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 041c692 by action🐍

@tdstein tdstein force-pushed the tdstein/this-is-the-way branch 2 times, most recently from 5dc33d8 to befb985 Compare April 2, 2025 02:18
@tdstein tdstein force-pushed the tdstein/this-is-the-way branch from befb985 to 041c692 Compare April 2, 2025 02:24
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.

1 participant