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

feat: truncate the labels before displaying in logs #550

Merged
merged 1 commit into from
Jun 20, 2023
Merged

Conversation

Charles546
Copy link
Collaborator

Description

The label value could be the complete output of a job. Better to shrink it before showing in the logs

This PR fixes the following issues

The label value could be the complete output of a job. Better to shrink it before showing in the logs
@Charles546 Charles546 requested a review from a team as a code owner June 16, 2023 03:25
@Charles546 Charles546 merged commit 478fc7c into dev Jun 20, 2023
9 checks passed
@Charles546 Charles546 deleted the CH-log branch June 20, 2023 17:04
dipperuser pushed a commit that referenced this pull request Jun 20, 2023
# [2.15.0](v2.14.0...v2.15.0) (2023-06-20)

### Bug Fixes

* bypass go-git unstaged change with reset ([#549](#549)) ([9f3a1ad](9f3a1ad))
* **deps:** update google.golang.org/genproto digest to e85fd2c ([#544](#544)) ([e20add8](e20add8))
* **deps:** update module github.com/gin-gonic/gin to v1.9.1 [security] ([#543](#543)) ([7d7dfe1](7d7dfe1))
* **deps:** update module github.com/imdario/mergo to v0.3.16 ([#545](#545)) ([3c41e5b](3c41e5b))
* load each context only once ([#546](#546)) ([1060096](1060096))

### Features

* support layered local variable definitions ([#547](#547)) ([2df21fc](2df21fc))
* truncate the labels before displaying in logs ([#550](#550)) ([478fc7c](478fc7c))
* web driver use token sources for authorization ([#548](#548)) ([545b736](545b736))
@dipperuser
Copy link
Collaborator

🎉 This PR is included in version 2.15.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants