Skip to content

Fix linter for docs#112

Open
vmercierfr wants to merge 1 commit into
qonto:mainfrom
vmercierfr:fix-linter-docs
Open

Fix linter for docs#112
vmercierfr wants to merge 1 commit into
qonto:mainfrom
vmercierfr:fix-linter-docs

Conversation

@vmercierfr

@vmercierfr vmercierfr commented Jun 6, 2026

Copy link
Copy Markdown
Collaborator

Goal

Fix linter

Why

In #110, we introduced code that is not compliant with linter. The linter workflow currently don't run on pull request so it was not detected.

Fork pull request workflows from outside collaborators of Github actions practices provides a way to run the workflow securely; we will address this in #113

How

  • Fix codegen code to comply with linter (constant, blank lines, ...)

Release

No release

Signed-off-by: Vincent Mercier <vmercier@gmail.com>
@vmercierfr vmercierfr requested review from dcupif, jbourdale and qfritz June 6, 2026 06:56
@vmercierfr vmercierfr added the go Pull requests that update Go code label Jun 6, 2026

@qfritz qfritz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why is the linter not running in your PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants