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

ci: Fixing misleading statuses when no updates on nightly build #8915

Closed
wants to merge 2 commits into from

Conversation

lewisd5
Copy link

@lewisd5 lewisd5 commented Jan 15, 2025

Wrapping the release commit in a git status (porcelain to remove human readable elements and no untracked files to avoid dotfiles impacting)

The steps following this should either auto skip or complete quickly allowing the workflow to finish quickly

https://ledgerhq.atlassian.net/browse/LIVE-12391

Copy link

vercel bot commented Jan 15, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
native-ui-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 15, 2025 4:39pm
react-ui-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 15, 2025 4:39pm
web-tools ❌ Failed (Inspect) Jan 15, 2025 4:39pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
ledger-live-github-bot ⬜️ Ignored (Inspect) Visit Preview Jan 15, 2025 4:39pm

@live-github-bot live-github-bot bot added the automation CI/CD stuff label Jan 15, 2025
@lewisd5 lewisd5 changed the title ci: checking git status before commit to avoid erroneous build failures ci: Fixing misleading statuses when no updates on nightly build Jan 15, 2025
@lewisd5 lewisd5 closed this Jan 16, 2025
@lewisd5 lewisd5 deleted the support/fixing-build-error-on-ni-change branch January 16, 2025 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation CI/CD stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant