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

Remove the hg up null workaround as it is no longer needed #69

Open
marco-c opened this issue Aug 4, 2021 · 3 comments
Open

Remove the hg up null workaround as it is no longer needed #69

marco-c opened this issue Aug 4, 2021 · 3 comments

Comments

@marco-c
Copy link
Collaborator

marco-c commented Aug 4, 2021

The new workaround from https://bugzilla.mozilla.org/show_bug.cgi?id=1720302 is much faster than hg up null we introduced in #68.

The workaround script lives at https://gist.github.com/zzzeid/46f7d8cc794f9fccf999bb8356f981be.

@abpostelnicu
Copy link
Collaborator

@marco-c if the current solution works, why don't we wait for the final patch?

@marco-c
Copy link
Collaborator Author

marco-c commented Aug 5, 2021

Yeah, maybe better to wait at this point. We can decide to use this new workaround in case we see the bot being too slow because of hg up null.

@marco-c
Copy link
Collaborator Author

marco-c commented Sep 7, 2021

The repo was fixed, so we can actually remove the workaround.

@marco-c marco-c changed the title Replace hg up null workaround with the new workaround from https://bugzilla.mozilla.org/show_bug.cgi?id=1720302 Remove the hg up null workaround as it is no longer needed Sep 7, 2021
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

2 participants