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

New blog post does not show up in fleek build preview if commit was amended #526

Open
meandavejustice opened this issue Jan 19, 2023 · 2 comments
Labels
need/triage Needs initial labeling and prioritization

Comments

@meandavejustice
Copy link
Contributor

It appears that blog posts are not built properly in ci if the commit was amended. This is currently happening with #510 and happened previously with #509, to solve last time I opened a fresh duplicate pr, #520. I don't want to keep cluttering up the pr queue with these duplicate prs 😬.

This may be an upstream issue with Fleek's workflow actions https://github.com/FleekHQ/workflows, but logging here until I get more info

@meandavejustice meandavejustice added the need/triage Needs initial labeling and prioritization label Jan 19, 2023
@vesahc
Copy link

vesahc commented Oct 5, 2023

Hi @meandavejustice, sorry I missed this. This has been an ongoing issue with many Fleek sites and I've been in contact with them to resolve it. We're making progress, but we still might see a few stuck PR's occasionally.

As you've figured out the best way is to create a new PR. There is another workaround that can unblock stuck builds in cases where we need to keep comments, approvals, etc. This does seem to only unblock the build previews for a handful of commits, sometimes requiring more updates to the "cloned" branch/PR to unblock them again.

I found a workaround! You need to create a new PR for the same commit using different branch name. This will kick the build upon completion both original and new PR will get status update. Then you can merge the original and close the new one.

If PR was created from remote repository you can pull the ref to your repository locally by following these instructions:

https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch origin pull/{id}/head:temporary

git push origin temporary

source: https://stackoverflow.com/questions/52200096/github-pull-request-waiting-for-status-to-be-reported

@vesahc
Copy link

vesahc commented Oct 20, 2023

According to Fleek support this should be resolved for all future PRs. Existing PRs may still be stuck and will still need to be "forced". I'm trying to keep an eye out for those, but if anyone gets blocked please ping me here or on Slack.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need/triage Needs initial labeling and prioritization
Projects
None yet
Development

No branches or pull requests

2 participants