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

Group log output on GitHub Actions #4961

Merged
merged 10 commits into from
Dec 12, 2023
Merged

Group log output on GitHub Actions #4961

merged 10 commits into from
Dec 12, 2023

Conversation

ferrarimarco
Copy link
Collaborator

@ferrarimarco ferrarimarco commented Dec 9, 2023

Fixes #2755

Proposed Changes

  1. Group log output using workflow commands
  2. Move RUN_LOCAL initialization early, so we can use it when logging.
  3. Create a log group for super-linter initialization.
  4. Create one log group for each linter.

Readiness Checklist

Author/Contributor

  • I included all the needed documentation for this change.
  • I provided the necessary tests.

Reviewing Maintainer

  • Label as breaking if this is a large, fundamental change.
  • Label as either: automation, bug, documentation, enhancement, infrastructure.

BEGIN_COMMIT_OVERRIDE
feat: group log output on GitHub Actions (#4961)
END_COMMIT_OVERRIDE

@ferrarimarco
Copy link
Collaborator Author

We need #4960 before this one.

@ferrarimarco ferrarimarco marked this pull request as draft December 11, 2023 21:16
@ferrarimarco ferrarimarco marked this pull request as ready for review December 11, 2023 21:32
zkoppert
zkoppert previously approved these changes Dec 12, 2023
Copy link
Contributor

@zkoppert zkoppert left a comment

Choose a reason for hiding this comment

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

👍🏻

* main:
  Reduce duplication in CI and CD workflows (#4982)
  Move instructions from the wiki to docs (#4957)
  Update documentation (#4981)
@ferrarimarco
Copy link
Collaborator Author

Pushing this one because I dismissed the review to solve merge conflicts. Thanks @zkoppert !

@ferrarimarco ferrarimarco merged commit 7150e1f into main Dec 12, 2023
3 checks passed
@ferrarimarco ferrarimarco deleted the group-output branch December 12, 2023 19:57
ferrarimarco added a commit that referenced this pull request Dec 12, 2023
* main:
  Group log output on GitHub Actions (#4961)
  Reduce duplication in CI and CD workflows (#4982)
  Move instructions from the wiki to docs (#4957)
  Update documentation (#4981)
  Bump com.puppycrawl.tools:checkstyle in /dependencies/checkstyle (#4966)
  Bump isort from 5.12.0 to 5.13.0 in /dependencies/python (#4967)
  Bump pylint from 3.0.2 to 3.0.3 in /dependencies/python (#4968)
  Bump cfn-lint from 0.83.3 to 0.83.4 in /dependencies/python (#4969)
  Bump npm-groovy-lint from 13.0.0 to 13.0.2 in /dependencies (#4970)
  Bump typescript from 5.3.2 to 5.3.3 in /dependencies (#4972)
  Bump next from 14.0.3 to 14.0.4 in /dependencies (#4973)
  Bump react-redux from 9.0.1 to 9.0.3 in /dependencies (#4974)
  Bump renovate from 37.83.5 to 37.89.7 in /dependencies (#4975)
  Bump prettier from 3.1.0 to 3.1.1 in /dependencies (#4976)
  Bump stylelint-scss from 5.3.1 to 5.3.2 in /dependencies (#4977)
  Bump golang from 1.21.4-alpine to 1.21.5-alpine (#4978)
  Bump actions/stale from 8 to 9 (#4980)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Group output per linter
2 participants