Skip to content

Conversation

lantoli
Copy link
Collaborator

@lantoli lantoli commented Jan 8, 2025

Initial file setup.

Link to any related issue(s): CLOUDP-293603

@@ -0,0 +1,107 @@
linters-settings:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

similar to TF repo one

@lantoli lantoli changed the title chore: Initial file setup choe: Initial file setup Jan 9, 2025
@lantoli lantoli changed the title choe: Initial file setup feat: Initial file setup Jan 9, 2025
@github-actions github-actions bot added the enhancement New feature or request label Jan 9, 2025
@lantoli lantoli removed the enhancement New feature or request label Jan 9, 2025
@lantoli lantoli changed the title feat: Initial file setup chore: Initial file setup Jan 9, 2025
@lantoli lantoli marked this pull request as ready for review January 9, 2025 08:24
"severity": "warning",
"pattern": [
{
"regexp": "^(?:\\x1b\\[\\d+m)?(.+?)(?:\\x1b\\[\\d+m)*:(?:\\x1b\\[\\d+m)*(\\d+)(?:\\x1b\\[\\d+m)*:(?:\\x1b\\[\\d+m)*(\\d+)(?:\\x1b\\[\\d+m)*: (?:\\x1b\\[\\d+m)*(.+?)(?:\\x1b\\[\\d+m)* \\[(.+?)\\]$",
Copy link
Contributor

Choose a reason for hiding this comment

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

What is this for?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

like in TF, it's for the actionlinter in code-health to show the errors inline in the files in GH UI, more info: https://github.com/actions/toolkit/blob/main/docs/problem-matchers.md

Copy link
Contributor

@EspenAlbert EspenAlbert left a comment

Choose a reason for hiding this comment

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

Nice!

@lantoli lantoli merged commit d58223d into main Jan 9, 2025
4 checks passed
@lantoli lantoli deleted the CLOUDP-293603_initial_setup branch January 9, 2025 13:29
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.

3 participants