Skip to content

Commit

Permalink
v3.0.0
Browse files Browse the repository at this point in the history
* Updated to Node 8
* Rewrote in Typescript
* Calling the library as a function has been removed in favor of `csv.parse`.
  * `csv()` change to `csv.parse()`
* Deprecated `fromString` in favor of `parseString`
  * `csv.fromString()` change to `csv.parseString()`
* Deprecated `fromStream` in favor of `parseStream`a
    * `csv.fromStream()` change to `csv.parseStream()`
* Deprecated`fromPath` in favor of `parseFile`
    * `csv.fromPath()` change to `csv.parseFile()`

# Conflicts:
#	.travis.yml
#	Gruntfile.js
#	History.md
#	lib/formatter/formatter.js
#	lib/formatter/formatter_stream.js
#	package-lock.json
#	package.json
  • Loading branch information
doug-martin committed Jun 5, 2019
1 parent d4e33a2 commit 011c960
Show file tree
Hide file tree
Showing 218 changed files with 352,488 additions and 1,208,908 deletions.
46 changes: 46 additions & 0 deletions .eslintrc.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
module.exports = {
parser: "@typescript-eslint/parser",
"parserOptions": {
"project": "./tsconfig.json"
},
plugins: ["@typescript-eslint"],
extends: ["airbnb-base", "plugin:@typescript-eslint/recommended"],
env: {
node: true,
mocha: true,
},
settings:{
"import/resolver": {
node: {
extensions: [".ts"]
}
}
},
globals: {
NodeJS: 'readonly',
},
rules: {
"indent": [
"error",
4
],
"comma-dangle": ["error", {
"arrays": "always-multiline",
"objects": "always-multiline",
"imports": "always-multiline",
"exports": "always-multiline",
"functions": "never"
}],
"no-restricted-syntax": ["error", "ForInStatement", "LabeledStatement", "WithStatement"],
"object-curly-spacing": ["error", "always"],
"array-bracket-spacing": ["error", "always"],
"no-underscore-dangle": 0,
"max-len": ["error", 150, 2, {
ignoreComments: false,
ignoreUrls: true,
ignoreRegExpLiterals: true,
ignoreStrings: true,
ignoreTemplateLiterals: true,
}]
},
};
30 changes: 30 additions & 0 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
---
name: Bug report
about: Create a report to help us improve
title: "[BUG]"
labels: bug
assignees: doug-martin, dustinsmith1024

---

**Describe the bug**
A clear and concise description of what the bug is.

**To Reproduce**
Steps to reproduce the behavior:
1. Example file contents if applicable
2. Example code to reproduce the issue.

**Expected behavior**
A clear and concise description of what you expected to happen.

**Screenshots**
If applicable, add screenshots to help explain your problem.

**Desktop (please complete the following information):**
- OS: [e.g. MacOS]
- OS Version [e.g. Mojave]
- Node Version [e.g. 10.16.0]

**Additional context**
Add any other context about the problem here.
20 changes: 20 additions & 0 deletions .github/ISSUE_TEMPLATE/feature_request.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
---
name: Feature request
about: Suggest an idea for this project
title: "[FEATURE]"
labels: enhancement
assignees: doug-martin, dustinsmith1024

---

**Is your feature request related to a problem? Please describe.**
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

**Describe the solution you'd like**
A clear and concise description of what you want to happen.

**Describe alternatives you've considered**
A clear and concise description of any alternative solutions or features you've considered.

**Additional context**
Add any other context or screenshots about the feature request here.
21 changes: 21 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
### All Submissions:

* [ ] Have you followed the guidelines in our Contributing document?
* [ ] Have you checked to ensure there aren't other open [Pull Requests](https://github.com/c2fo/fast-csv/pulls) for the same update/change?

<!-- You can erase any parts of this template not applicable to your Pull Request. -->

### New Feature Submissions:

1. [ ] Have you added tests for the new feature
2. [ ] Does your submission pass tests?
3. [ ] Have you lint your code locally prior to submission?
4. [ ] Have you updated the docs?
* [ ] If you added new parsing or formatting options have you added them to the docs?
* [ ] If applicable have you added an example to the parsing or formatting docs?

### Changes to Core Features:

* [ ] Have you added an explanation of what your changes do and why you'd like us to include them?
* [ ] Have you written new tests for your core changes, as applicable?
* [ ] Have you successfully ran tests with your changes locally?
7 changes: 4 additions & 3 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
node_modules
*.iml
.DS_Store
lib-cov
.idea
.grunt
atlassian-ide-plugin.xml
benchmark/results
.nyc_output
build
**/tmp
3 changes: 0 additions & 3 deletions .gitmodules

This file was deleted.

5 changes: 5 additions & 0 deletions .huskyrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
{
"hooks": {
"pre-commit": "npm test"
}
}
56 changes: 0 additions & 56 deletions .jshintrc

This file was deleted.

14 changes: 14 additions & 0 deletions .mocharc.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
module.exports = {
diff: true,
require: ['ts-node/register', 'source-map-support/register'],
extension: ['ts', 'js'],
package: './package.json',
reporter: 'spec',
slow: 75,
timeout: 2000,
recursive: true,
ui: 'bdd'
};

// --recursive
// test/
11 changes: 3 additions & 8 deletions .npmignore
Original file line number Diff line number Diff line change
@@ -1,13 +1,8 @@
benchmark
test
src
node_modules
docs
docs-md
support
lib-cov
Makefile
benchmark
.travis.yml
Gruntfile.js
History.md
examples

.nyc_output
1 change: 1 addition & 0 deletions .nvmrc
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
v8.16.0
20 changes: 20 additions & 0 deletions .nycrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
{
"extends": "@istanbuljs/nyc-config-typescript",
"all": true,
"check-coverage": true,
"exclude": [
"**/*.d.ts",
"coverage/**",
"packages/*/test/**",
"test/**",
"test{,-*}.ts",
"**/*{.,-}{test,spec}.ts",
"**/__tests__/**",
"**/node_modules/**",
"benchmark/**/*",
"examples/**/*",
"docs/**/*",
".eslintrc.js",
".mocharc.js"
]
}
12 changes: 4 additions & 8 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,11 +1,7 @@
language: node_js
node_js:
- "4"
- "5"
- "6"
- "7"
- "8"
- "node"
- "10"
- "stable"
before_script:
- npm install -g grunt-cli
- "8"

after_success: npm run coverage
57 changes: 0 additions & 57 deletions Gruntfile.js

This file was deleted.

20 changes: 20 additions & 0 deletions History.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,23 @@
# v3.0.0

* Updated to Node 8
* Rewrote in Typescript

## Parsing Changes
* Calling the library as a function has been removed in favor of `csv.parse`.
* `csv()` change to `csv.parse()`
* Deprecated `fromString` in favor of `parseString`
* `csv.fromString()` change to `csv.parseString()`
* Deprecated `fromStream` in favor of `parseStream`a
* `csv.fromStream()` change to `csv.parseStream()`
* Deprecated`fromPath` in favor of `parseFile`
* `csv.fromPath()` change to `csv.parseFile()`

## Formatting Changes
* `csv.createWriteStream` has been removed in favor of `csv.format`
* `csv.writeToBuffer` and `csv.writeToString` no longer accepts a `callback`, instead they return a promise `Promise`


# v2.5.0

* Use safer-buffer polyfill to support node >= 8.
Expand Down
2 changes: 1 addition & 1 deletion LICENSE
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
The MIT License

Copyright (c) 2011-2012 C2FO
Copyright (c) 2011-2019 C2FO

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
Expand Down
Loading

0 comments on commit 011c960

Please sign in to comment.