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.

Sign up for GitHub

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

Jump to bottom

add pre-push hook that will run linters and unit tests #744

Merged
merged 2 commits into from
Oct 20, 2023
Merged

add pre-push hook that will run linters and unit tests #744

merged 2 commits into from
Oct 20, 2023

Conversation

Tofel
Copy link
Contributor

@Tofel Tofel commented Oct 20, 2023

To enable run inside the CTF directory:

git config core.hooksPath .githooks

and what you'll see will look like this:
image

@cl-sonarqube-production
Copy link

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@Tofel Tofel merged commit fd1e8fd into main Oct 20, 2023
6 of 7 checks passed
@Tofel Tofel deleted the add_pre-push_hook branch October 20, 2023 16:34
Tofel added a commit that referenced this pull request Oct 24, 2023
@Tofel
add pre-push hook that will run linters and unit tests (#744)
b49ba4c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AnieeG AnieeG AnieeG approved these changes

@sebawo sebawo Awaiting requested review from sebawo sebawo is a code owner

@skudasov skudasov Awaiting requested review from skudasov

@kalverra kalverra Awaiting requested review from kalverra

@tateexon tateexon Awaiting requested review from tateexon

@gheorghestrimtu gheorghestrimtu Awaiting requested review from gheorghestrimtu

@lukaszcl lukaszcl Awaiting requested review from lukaszcl

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Tofel @AnieeG