actions-importer requires extraneous arguments?

[jrtitus]

Why are the DevOps organization and project names required when the source-file is local?

$ gh actions-importer dry-run azure-devops pipeline --source-file-path ./path/to/azure-pipelines-pr.yml --features ghes-latest --no-telemetry -o ./workflows
[2023-03-05 20:57:53] No value provided for the required options '--azure-devops-organization, --azure-devops-project'
[2023-03-05 20:57:53] Logs: './log/valet-20230305-205753.log'

NOTE: the log directory does not exist after this command fails.

Why are credentials required for github.com if the actions importer image is already available locally and dry-run is specified? I was under the impression that credentials were only required if issuing one of the update, audit, forecast, or migrate commands.

$ docker images | grep actions-importer
ghcr.io/actions-importer/cli         latest       7811041f3e86   2 days ago      215MB

$ gh actions-importer dry-run azure-devops pipeline --source-file-path ./path/to/azure-pipelines-pr.yml --features ghes-latest --no-telemetry -o ./workflows -g ORG_NAME -p PROJECT_NAME
[2023-03-05 20:59:14] Unable to locate credentials for 'https://github.com'
[2023-03-05 20:59:14] Logs: './workflows/log/valet-20230305-205914.log'

NOTE: the workflows/log directory does exist after this command fails

# Logfile created on 2023-03-05 21:15:40 +0000 by logger.rb/v1.5.3
I, [2023-03-05T21:15:40.923354 #1]  INFO -- : Using GitHub Features: 'ghes-latest'
E, [2023-03-05T21:15:40.923395 #1] ERROR -- : Unable to locate credentials for 'https://github.com'
/usr/local/bundle/gems/actions_importer-1.1.16676/lib/valet/models/credentials_store.rb:79:in `credentials_for_url'
/usr/local/bundle/gems/actions_importer-1.1.16676/lib/valet/models/credentials_store.rb:74:in `block in parse_credentials'
/usr/local/bundle/gems/actions_importer-1.1.16676/lib/valet/models/credentials_store.rb:70:in `each'
/usr/local/bundle/gems/actions_importer-1.1.16676/lib/valet/models/credentials_store.rb:70:in `parse_credentials'
/usr/local/bundle/gems/actions_importer-1.1.16676/lib/valet/concerns/cli_utilities.rb:68:in `fill_valet_options'
/usr/local/bundle/gems/actions_importer-1.1.16676/lib/valet/subcommands/azure_devops/dry_run.rb:35:in `call'
/usr/local/bundle/gems/actions_importer-1.1.16676/lib/valet/subcommands/azure_devops/dry_run.rb:17:in `pipeline'
/usr/local/bundle/gems/thor-1.2.1/lib/thor/command.rb:27:in `run'
/usr/local/bundle/gems/thor-1.2.1/lib/thor/invocation.rb:127:in `invoke_command'
/usr/local/bundle/gems/thor-1.2.1/lib/thor.rb:392:in `dispatch'
/usr/local/bundle/gems/thor-1.2.1/lib/thor/invocation.rb:116:in `invoke'
/usr/local/bundle/gems/thor-1.2.1/lib/thor.rb:243:in `block in subcommand'
/usr/local/bundle/gems/thor-1.2.1/lib/thor/command.rb:27:in `run'
/usr/local/bundle/gems/thor-1.2.1/lib/thor/invocation.rb:127:in `invoke_command'
/usr/local/bundle/gems/thor-1.2.1/lib/thor.rb:392:in `dispatch'
/usr/local/bundle/gems/thor-1.2.1/lib/thor/invocation.rb:116:in `invoke'
/usr/local/bundle/gems/thor-1.2.1/lib/thor.rb:243:in `block in subcommand'
/usr/local/bundle/gems/thor-1.2.1/lib/thor/command.rb:27:in `run'
/usr/local/bundle/gems/thor-1.2.1/lib/thor/invocation.rb:127:in `invoke_command'
/usr/local/bundle/gems/thor-1.2.1/lib/thor.rb:392:in `dispatch'
/usr/local/bundle/gems/thor-1.2.1/lib/thor/base.rb:485:in `start'
/usr/local/bundle/gems/actions_importer-1.1.16676/exe/actions-importer:19:in `<top (required)>'
/usr/local/bundle/bin/actions-importer:25:in `load'
/usr/local/bundle/bin/actions-importer:25:in `<main>'

[assyadh]

Hi jritus,

Unfortunately right now the cli do not discriminate for which command you are running (audit vs dry run for example), hence we cannot make cli options conditional and they are required today.

With regards to the azure devops required arguments, they are required and used to construct the output filepath. You can use a placeholder for that if supplying actual values is not possible right now.