Skip to content

Conversation

@jan--f
Copy link
Collaborator

@jan--f jan--f commented May 14, 2024

This fixes commitlint failures caused by
wagoid/commitlint-github-action#763.

@jan--f jan--f requested a review from a team as a code owner May 14, 2024 13:24
@jan--f jan--f requested review from JoaoBraveCoding and danielmellado and removed request for a team May 14, 2024 13:24
@openshift-ci openshift-ci bot requested a review from simonpasquier May 14, 2024 13:24
@openshift-ci
Copy link

openshift-ci bot commented May 14, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jan--f

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

And update commitlint action to v6.
This fixes commitlint failures caused by
wagoid/commitlint-github-action#763.

Signed-off-by: Jan Fajerski <[email protected]>
@jan--f jan--f force-pushed the fix-commitlint-action-cfg-extension branch from bdaf30a to 9d2f6eb Compare May 14, 2024 13:34
Copy link
Contributor

Choose a reason for hiding this comment

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

We would need to change module.exports to export default Configuration; in here to comply with the ES module syntax.

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 so?

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah, I think that should make it.

@danielmellado
Copy link
Contributor

/lgtm

};

module.exports = Configuration;
export default Configuration
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
export default Configuration
export default Configuration;

Copy link
Contributor

Choose a reason for hiding this comment

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

let's keep the end trailing even if it does seem that it works without it.

@openshift-merge-bot openshift-merge-bot bot merged commit 33584e6 into rhobs:main May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants