-
Notifications
You must be signed in to change notification settings - Fork 150
NO-SNOW fix tiny typo in DataTypes.md #1153
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.
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
base: master
Are you sure you want to change the base?
Conversation
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) ✅ license/snyk check is complete. No issues have been found. (View Details) |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1153 +/- ##
==========================================
- Coverage 87.64% 87.58% -0.06%
==========================================
Files 133 133
Lines 12818 12818
Branches 1320 1320
==========================================
- Hits 11234 11227 -7
- Misses 1260 1265 +5
- Partials 324 326 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
doc/DataTypes.md
Outdated
| `bool` | `BOOLEAN` | | ||
| `byte` | `BINARY` | | ||
| `datetime` | `DATE` | | ||
| .NET Framework Data Type | Data Type in Snowflake | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe just .NET Data type
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, modified
Code isn't changed, this is a documentation update.
This is a very small negligible issue which was accidentally discovered on one of the Support weekly calls, but since it was discovered, why not fix it.