Skip to content

Dependabot sends PRs to forks #777

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

Closed
jakirkham opened this issue Jun 16, 2021 · 14 comments
Closed

Dependabot sends PRs to forks #777

jakirkham opened this issue Jun 16, 2021 · 14 comments

Comments

@jakirkham
Copy link
Member

Just ran into this personally ( https://github.com/jakirkham/zarr-python/pull/1 ). Looks like this is an open upstream issue ( dependabot/dependabot-core#2804 )

@joshmoore
Copy link
Member

@jakirkham : did this start following updating your main branch? Did you see any workaorunds that would could potentially work for you, or shall we just disable it until this passes?

@jakirkham
Copy link
Member Author

Yeah. Not that I could see. I'm not sure

@jakirkham
Copy link
Member Author

Actually I guess their recommendation is delete the fork and refork. Not ideal, but maybe worth trying

@jakirkham
Copy link
Member Author

Ok took the plunge and deleted my fork and reforked. Hopefully I haven't broken things too badly on my end. Will see if that fixes it

@joshmoore
Copy link
Member

Wow. Thanks for taking one for the team. Keep us posted.

@jakirkham
Copy link
Member Author

IPython helped make it easier to sync everything locally 😄

No PRs yet 🍀

@jakirkham
Copy link
Member Author

Haven't seen PRs in many months. So I think the answer is "delete and refork". Not ideal, but not much we can do about it either. Users are also free to raise this problem upstream ( dependabot/dependabot-core#2804 ). Going to close.

@jakirkham jakirkham reopened this Oct 18, 2022
@jakirkham
Copy link
Member Author

Unfortunately have started seeing this with Numcodecs. Tried deleting the fork and reforking, but the issue still occurs

jakirkham/numcodecs#1

@jakirkham
Copy link
Member Author

Ok tried deleting the fork again and reforking. It seems to be working. Will keep an eye on it.

Sadly this closed and orphaned existing PRs from my repo.

We may want to revisit dependabot support given this issue remains unaddressed (and may dissuade new contributors from engaging).

@joshmoore
Copy link
Member

dependabot/dependabot-core#2804 (comment)

We're actively working on this and a permanent fix should land in the next couple of weeks. We'll share more around the time we''re ready to roll this out

🤞🏽

@jakirkham
Copy link
Member Author

Yep saw that. Though that comment is from a couple weeks back

@joshmoore
Copy link
Member

In the mean time, do we need to restore these?

@jakirkham
Copy link
Member Author

Tried reopening, but they are orphaned. One can still retrieve the code from them

@jhamman
Copy link
Member

jhamman commented Oct 17, 2024

I have not been seeing this recently so I think this must be a thing of the past.

@jhamman jhamman closed this as completed Oct 17, 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

No branches or pull requests

3 participants