Skip to content
This repository has been archived by the owner on Aug 16, 2022. It is now read-only.

fix: import dataset from google sheet bug #23

Merged
merged 3 commits into from Jul 14, 2021

Conversation

yk-eukarya
Copy link
Contributor

Overview

What I've done

Check if the column has an empty label => ignore it

Note

This will affect the import from file also in the same way

What I haven't done

How I tested

Which point I want you to review particularly

Memo

@yk-eukarya yk-eukarya added the bug Something isn't working label Jul 13, 2021
@yk-eukarya yk-eukarya self-assigned this Jul 13, 2021
@codecov
Copy link

codecov bot commented Jul 13, 2021

Codecov Report

Merging #23 (f5d4941) into main (a2059e9) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #23   +/-   ##
=======================================
  Coverage   16.00%   16.00%           
=======================================
  Files         198      198           
  Lines       40002    40002           
=======================================
  Hits         6401     6401           
  Misses      32633    32633           
  Partials      968      968           
Impacted Files Coverage Δ
pkg/dataset/csvparser.go 70.00% <100.00%> (ø)

@HideBa HideBa merged commit 077558c into main Jul 14, 2021
@HideBa HideBa deleted the fix/ImportDatasetFromGoogleSheetBug branch July 14, 2021 01:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
4 participants