Skip to content
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

Keep GitHub Actions up to date with GitHub's Dependabot #416

Closed

Conversation

cclauss
Copy link

@cclauss cclauss commented Aug 13, 2024

Fixes software supply chain safety warnings like at the bottom right of
https://github.com/python-greenlet/greenlet/actions/runs/10376229051

Also, Python 3.7 was already end-of-life when Apple Silicon was released so Python 3.7 is not supported on modern Macs.

@cclauss cclauss force-pushed the keep-GitHub-Actions-up-to-date branch from 0dd5c75 to 9f9a585 Compare August 13, 2024 20:48
@cclauss cclauss force-pushed the keep-GitHub-Actions-up-to-date branch from 0e0d1c9 to bbc0018 Compare August 13, 2024 21:34
Copy link

@EwoutH EwoutH left a comment

Choose a reason for hiding this comment

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

Looks good!

@cclauss
Copy link
Author

cclauss commented Aug 18, 2024

Do you know how to correct the Linux tests?

@EwoutH
Copy link

EwoutH commented Aug 18, 2024

From reading the error, you might need to mark it as safe.

git config --global --add safe.directory /greenlet/.git

I think you can just add a step:

    - name: Configure Git safe directory
      run: git config --global --add safe.directory /greenlet/.git

@cclauss
Copy link
Author

cclauss commented Aug 18, 2024

I did hack with that but was unsuccessful. Will try again on a separate PR.

@jamadden jamadden closed this in 81bba3c Sep 9, 2024
@cclauss cclauss deleted the keep-GitHub-Actions-up-to-date branch September 10, 2024 06:01
@cclauss
Copy link
Author

cclauss commented Sep 10, 2024

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