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

[Merged by Bors] - Linkcheck fix #3452

Closed
wants to merge 2 commits into from

Conversation

realbigsean
Copy link
Member

Issue Addressed

I think we're running into this in our linkcheck, so I'm going to frist verify linkcheck fails on the current version, and then try downgrading it to see if it passes chronotope/chrono#755

@realbigsean
Copy link
Member Author

Oh I think we need to change the lighthouse book for the linkcheck to run in CI.. but this worked for me locally

@realbigsean realbigsean added ready-for-review The code is ready for review low-hanging-fruit Easy to resolve, get it before someone else does! labels Aug 10, 2022
Copy link
Member

@michaelsproul michaelsproul left a comment

Choose a reason for hiding this comment

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

Happy to try this, although I don't see how the chrono issue is implicated

@realbigsean
Copy link
Member Author

image

The panic is here, this is from the gas limit API CI run

@realbigsean
Copy link
Member Author

The mdbook docker container uses chrono, was digging into it earlier. Sorry for the lack of description 😊

@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Aug 11, 2022
@michaelsproul
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Aug 11, 2022
## Issue Addressed

I think we're running into this in our linkcheck, so I'm going to frist verify linkcheck fails on the current version, and then try downgrading it to see if it passes chronotope/chrono#755

Co-authored-by: realbigsean <sean@sigmaprime.io>
@bors
Copy link

bors bot commented Aug 11, 2022

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Aug 11, 2022
## Issue Addressed

I think we're running into this in our linkcheck, so I'm going to frist verify linkcheck fails on the current version, and then try downgrading it to see if it passes chronotope/chrono#755

Co-authored-by: realbigsean <sean@sigmaprime.io>
@bors bors bot changed the title Linkcheck fix [Merged by Bors] - Linkcheck fix Aug 11, 2022
@bors bors bot closed this Aug 11, 2022
@realbigsean realbigsean deleted the linkcheck-fix branch November 21, 2023 16:15
Woodpile37 pushed a commit to Woodpile37/lighthouse that referenced this pull request Jan 6, 2024
## Issue Addressed

I think we're running into this in our linkcheck, so I'm going to frist verify linkcheck fails on the current version, and then try downgrading it to see if it passes chronotope/chrono#755

Co-authored-by: realbigsean <sean@sigmaprime.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low-hanging-fruit Easy to resolve, get it before someone else does! ready-for-merge This PR is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants