feat: add actor_location column to the github_org_audit_log table

[patrickdevivo]

Adds a new column actor_location to the github_org_audit_log

[codecov]

Codecov Report

Merging #296 (b08d715) into main (58f2c6d) will decrease coverage by 0.04%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##             main     #296      +/-   ##
==========================================
- Coverage   71.69%   71.65%   -0.05%     
==========================================
  Files          57       57              
  Lines        3466     3471       +5     
==========================================
+ Hits         2485     2487       +2     
- Misses        620      621       +1     
- Partials      361      363       +2     

Impacted Files	Coverage Δ
extensions/internal/github/org_audit_log.go	69.00% <40.00%> (-1.53%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 58f2c6d...b08d715. Read the comment docs.

[amenowanna]

Is there a reason we are choosing JSON vs JSONB?

[patrickdevivo]

Is there a reason we are choosing JSON vs JSONB?

So SQLite doesn't really have a concept of JSONB - in fact, JSON isn't a "real" datatype in SQLite either (https://www.sqlite.org/datatype3.html), the type system is pretty flexible. Here, we just mark the column as JSON to indicate to the user to expect a string that can be treated as JSON.

There are some built-in JSON functions/operators that allow for queries such as:

select json_extract(actor_location, '$.city'), * from github_org_audit_log('mergestat')

[amenowanna]

Is there a reason we are choosing JSON vs JSONB?

So SQLite doesn't really have a concept of JSONB - in fact, JSON isn't a "real" datatype in SQLite either (https://www.sqlite.org/datatype3.html), the type system is pretty flexible. Here, we just mark the column as JSON to indicate to the user to expect a string that can be treated as JSON.

There are some built-in JSON functions/operators that allow for queries such as:

select json_extract(actor_location, '$.city'), * from github_org_audit_log('mergestat')

TIL. Thank you!