Skip to content

Sonarcube issue fix solution test #104

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

Open
MagnusEngh wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Sonarcube issue fix solution test #104

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3ed1054
Testing fix, now make fix
MagnusEngh Jul 1, 2026
451841c
Merge branch 'main' into sonarcube-issue-fix-solution-test
MagnusEngh Jul 1, 2026
cc25709
Testing
MagnusEngh Jul 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 8 additions & 4 deletions src/sonarcube_fixes/test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,13 @@
import duckdb as db

navn = "eksempel"
splitt_7172 = db.sql(
db.sql(
f"""select distinct {navn}, verdi as rente from r12_koblet2 where art in ('71','72')"""
)
splitt_0160 = db.sql(
).create_view("splitt_7172")
db.sql(
f"""select distinct {navn}, verdi as belop_50 from r12_koblet2 where art in ('01','60')"""
)
).create_view("splitt_0160")

sp1=db.sql(f"""select a.*, b.belop_50 from splitt_7172 a full join splitt_0160 b
on a.periode=b.periode and a.selskap=b.selskap and a.serienavn=b.serienavn and a.ltid=b.ltid and a.sek=b.sek
where a.rente is not null and b.belop_50 is not null""").df()

Check warning on line 13 in src/sonarcube_fixes/test.py

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Add replacement fields or use a normal string instead of an f-string. 


See more on https://sonarcloud.io/project/issues?id=statisticsnorway_tech-coach-examples&issues=AZ8cemCztlF8EVe9OhRX&open=AZ8cemCztlF8EVe9OhRX&pullRequest=104
Loading