Add unseen columns to informant upon `incorporate()`
Open
@rich-iannone is already working on this.
Since Sep 19, 2020.
Difficulty: [3] Advanced
Effort: [3] High
Priority: [3] High
Type: ★ Enhancement
- Dominant language
- R
- Stars
- 1k
- Forks
- 59
- Avg merge
- 25m
- Merged PRs (30d)
- 4
Description
As tables may change between uses of incorporate(), the informant currently only updates dimension values (column and row counts) but doesn’t update the columns section with the names of new columns. This should be done, perhaps updating the order of columns as well.
On top of this, some columns from a target table might be dropped or renamed over time. We need some ways to handle these changes without losing information. That’ll be a different issue though.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.