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

chore: rm upstream-py27 #564

Merged
merged 1 commit into from
Feb 23, 2023
Merged

Conversation

texastony
Copy link
Contributor

Issue #, if available: Pyca has dropped 2.7 support, we do not need to maintain test/upstream-requirements-py27.txt.

Description of changes: rm test/upstream-requirements-py27.txt

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Check any applicable:

  • Were any files moved? Moving files changes their URL, which breaks all hyperlinks to the files.

@texastony texastony requested a review from a team as a code owner February 22, 2023 23:39
@texastony
Copy link
Contributor Author

texastony commented Feb 23, 2023

CI failed due to GitHub throwing a 429,
i.e: Too many CI actions tried to pull from the repo and GitHub throttled us.
Too prevent this in the future,
we should consolidate are GitHub Workflows.

Technically, a workflow could run all the tox envs for a OS Version,
even across Python versions.

That would dramatically reduce the requests to pull the repo down.

I am not going to address that in this PR.
But I will either cut us ticket or GitHub issue.

Instead, I will just wait until the AWS CodeBuild test passes,
and then re-run the failed GitHub actions only.

@texastony texastony merged commit b378508 into aws:master Feb 23, 2023
@texastony texastony deleted the tony/remove-upstream-py27 branch February 23, 2023 00:37
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