commercetest / commercetest/nlnet

Move the `add_explanations` function to the script `export_to_rdf.py`

Open
#74 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
1
Forks
0
PR merge metrics
No merged PRs in 30d

Description

This is created as per the review in the PR : #68
Review:
It might be helpful to provide a summary of which column(s) in the row contained unexpected null values. Otherwise users of the script don't know what data needs adding to the source data (the TSV file) to remove/replace the nulls with suitable values.

Contributor guide

No contributing guide indexed for this repository

Research direction

Review PR #68 and locate the existing add_explanations function, then inspect export_to_rdf.py to determine where it belongs. Done means the function is moved into that script and the unexpected-null explanation, including the affected column summary described in the review, remains covered.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data-engineering
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.