8000 Change name to redox_parser by taylor-cedar · Pull Request #29 · cedar-team/pyredox · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Change name to redox_parser #29

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.

Sign up for GitHub

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

Jump to bottom
Op 8000 en
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Change name to redox_parser #29

wants to merge 1 commit into from

Conversation

taylor-cedar
Copy link

No description provided.

@mmabey
Copy link
Contributor
mmabey commented Aug 28, 2023

The errors you're encountering are ones I've already developed fixes for. I understand if you don't want my help, but I'm offering it just the same.

The test won't pass because the workflow action that installs Poetry is using an outdated version of Node that GitHub dropped legacy support for some time ago. I was able to get everything working by forking and using my own version of the action: https://github.com/mmabey/action-setup-python

The changes from the base are very minimal, just some Dependabot PRs and updating the version of actions/cache that uses a newer version of Node. It would be pretty trivial for you to fork your own version of that action and make the same updates if you're uncomfortable using my repo.

I know we have our differences in opinion of how to best maintain this library going forward, but I honestly only have the library's interest at heart. If you ever feel comfortable, please don't hesitate to use me as a resource in your work on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@taylor-cedar @mmabey
0