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

fix: improve CI output #52

Merged
merged 2 commits into from
Oct 29, 2024
Merged

fix: improve CI output #52

merged 2 commits into from
Oct 29, 2024

Conversation

mdonnalley
Copy link
Contributor

@mdonnalley mdonnalley commented Oct 24, 2024

  • add onlyShowAtEndInCI config prop so that consumers can choose for information to only be shown at the very end when in CI
  • Don't repeat stage headers when receiving updates to stage specific infos
  • Reprint stage or stage specific infos after 5 minutes (configurable with OCLIF_CI_HEARTBEAT_FREQUENCY_MS or SF_CI_HEARTBEAT_FREQUENCY_MS)
  • Throttle new information every 5 seconds (configurable with OCLIF_CI_UPDATE_FREQUENCY_MS or SF_CI_UPDATE_FREQUENCY_MS)

forcedotcom/cli#3079
@W-17060727@

@mdonnalley mdonnalley changed the title fix: never repeat current stage in CI mode fix: improve CI output Oct 25, 2024
@cristiand391
Copy link
Member

QA notes:

setup:
tested a few via bin/run.js project deploy start on dreamhouse and the sf-specific deploy example in this repo (mostly to test the new features with bunch of updates on multiple stages).

✅ stage headers aren't duplicated on data updates
✅ can configure reprint stage info via SF_CI_HEARTBEAT_FREQUENCY_MS
SF_CI_HEARTBEAT_FREQUENCY_MS lower than the deploy poll interval, I can see previous data being printed, if new data comes in the frequency is still respected
SF_CI_UPDATE_FREQUENCY_MS throttles new info frequency, setting it to 10000 on a small deploy I only see the start and the finished update after 10s
✅ stages with onlyShowAtEndInCI are rendered at the end of stages when it finished.

@cristiand391 cristiand391 merged commit d3bb362 into main Oct 29, 2024
11 checks passed
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

Successfully merging this pull request may close these issues.

2 participants