drivendataorg / drivendataorg/nbautoexport

Proposal: additive `configure`

Open
#66 2 comments 1 reaction 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
87
Forks
10
PR merge metrics
No merged PRs in 30d

Description

* nbautoexport version: 0.2.1
* Python version: 3.9.1
* Operating System: Ubuntu 18.04.1

### Description

In the `clean` doc: https://nbautoexport.drivendata.org/cleaning/
I tried to add a folder to exclude, and it overwrote my old non-default `export_formats` config.
I think it would be more natural to not have everything overwritten.

### What I Did

```
❯ cat notebooks/.nbautoexport
{
"export_formats": [
"script",
"html"
],
"organize_by": "extension",
}
❯ nbautoexport configure notebooks/ \
--clean-exclude 'report/*'
Detected existing autoexport configuration at notebooks/.nbautoexport. If you wish to overwrite, use the --overwrite flag.
❯ nbautoexport configure notebooks/ --overwrite \
--clean-exclude 'report/*'
❯ cat notebooks/.nbautoexport
{
"export_formats": [
"script"
],
"organize_by": "extension",
"clean": {
"exclude": [
"report/*"
]
}
}
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.