Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[FEATURE] Add CLI command to migrate a folder of Python files and notebooks on local file system #3514

Open
Tracked by #1398
JCZuurmond opened this issue Jan 13, 2025 · 0 comments · May be fixed by #3520
Open
Tracked by #1398

[FEATURE] Add CLI command to migrate a folder of Python files and notebooks on local file system #3514

JCZuurmond opened this issue Jan 13, 2025 · 0 comments · May be fixed by #3520
Assignees
@JCZuurmond
Labels
migrate/code Abstract Syntax Trees and other dark magic migrate/python Pull requests that update Python code

Comments

@JCZuurmond
Copy link
Member

JCZuurmond commented Jan 13, 2025 •
edited
Loading

Description

Add CLI command to migrate a folder of files/notebooks on local file system
@JCZuurmond JCZuurmond mentioned this issue Jan 13, 2025
Open
3 tasks
@github-project-automation github-project-automation bot moved this to Todo in UCX Jan 13, 2025
@JCZuurmond JCZuurmond added migrate/code Abstract Syntax Trees and other dark magic migrate/python Pull requests that update Python code labels Jan 13, 2025
@JCZuurmond JCZuurmond changed the title Add CLI command to migrate a folder of files/notebooks on local FS [FEATURE] Add CLI command to migrate a folder of files/notebooks on local file system Jan 13, 2025
@JCZuurmond JCZuurmond changed the title [FEATURE] Add CLI command to migrate a folder of files/notebooks on local file system [FEATURE] Add CLI command to migrate a folder of Python files and notebooks on local file system Jan 13, 2025
@JCZuurmond JCZuurmond mentioned this issue Jan 13, 2025
Merged
@JCZuurmond JCZuurmond self-assigned this Jan 13, 2025
@JCZuurmond JCZuurmond moved this from Todo to In Progress in UCX Jan 13, 2025
This was referenced Jan 14, 2025
Draft
Merged
Merged
Open
github-merge-queue bot pushed a commit that referenced this issue Jan 17, 2025
@JCZuurmond
Delete stale code: NotebookLinter._load_source_from_run_cell (#3529)
29743d9 
## Changes
Delete stale code: `NotebookLinter._load_source_from_run_cell`

### Linked issues

Progresses #3514

### Functionality

- [x] modified code linting related
- [x] modified existing command: `databricks labs ucx lint-local-code`

### Tests

- [x] manually tested
- [x] added and modified unit tests
github-merge-queue bot pushed a commit that referenced this issue Jan 17, 2025
@JCZuurmond
Rename Python AST's Tree methods for clarity (#3524)
f47722c 
## Changes
Rename Python AST's `Tree` methods for clarity

- Add docstrings
- Rename `append_` to `attach_` and `extend_` to more precisely describe
what the methods do
- Let `attach_` and `extend_` always return `None`
- Extend unit testing

### Linked issues

Progresses #3514
Precedes #3520

### Functionality

- [x] modified code linting related
- [x] modified existing command: `databricks labs ucx lint-local-code`

### Tests

- [x] manually tested
- [x] added and modified unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JCZuurmond JCZuurmond

Labels
migrate/code Abstract Syntax Trees and other dark magic migrate/python Pull requests that update Python code
Projects
UCX
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix cli command to migrate local folder databrickslabs/ucx
1 participant
@JCZuurmond