Skip to content

Make --config accept globs#5090

Merged
dbkr merged 7 commits intodevelopfrom
dbkr/deploy_config_glob
Sep 21, 2017
Merged

Make --config accept globs#5090
dbkr merged 7 commits intodevelopfrom
dbkr/deploy_config_glob

Conversation

@dbkr
Copy link
Copy Markdown
Member

@dbkr dbkr commented Sep 20, 2017

So by default it will symlink all config files in the current dir

Builds on #5089 (easier to just keep everything in this PR)

and make the default config*.sjon so we'll automatically symlink
any config files in the current dir.
because that's really all it's doing and we can use it for other
things
Because redeploy.py already has a 'symlink' option that does
something else.
Copy link
Copy Markdown
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

generally looks sane

Comment thread scripts/deploy.py Outdated
for include in args.include:
deployer.symlink_paths.update({ os.path.basename(pth): pth for pth in glob.iglob(include) })

print("%r", (deployer.symlink_paths,))
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks like leftover debug?

Comment thread scripts/redeploy.py Outdated
deployer.should_clean = args.clean
deployer.config_locations = dict(args.configs) if args.configs else {}

deployer.symlink_paths = {}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't this the default?

@richvdh richvdh assigned dbkr and unassigned richvdh Sep 20, 2017
Comment thread scripts/deploy.py Outdated
"config.json": args.config,
}

deployer.symlink_paths = {}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also here, sorry

Copy link
Copy Markdown
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm otherwise

@dbkr dbkr merged commit 26ec25b into develop Sep 21, 2017
@t3chguy t3chguy deleted the dbkr/deploy_config_glob branch May 12, 2022 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants