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.

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

Add -w workspace and -d directory flags to plan/apply comments #14

Merged
merged 4 commits into from
Feb 26, 2018

Conversation

lkysow
Copy link
Member

@lkysow lkysow commented Feb 26, 2018

Now you can specify which workspace and directory to plan/apply in via

atlantis plan -d dir/child -w staging

Also, any additional options to proxy to terraform must be specified after a --:

atlantis plan -w staging -- -target=resource

Fixes #12

Ensure -d flag uses relative dirs and doesn't allow for
directory traversal.

Fix bug where if there was an error in PreExecute, we
wouldn't unlock, leaving a possibly abandoned lock.
@codecov
Copy link

codecov bot commented Feb 26, 2018

Codecov Report

Merging #14 into master will increase coverage by <.01%.
The diff coverage is 70.78%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage   63.77%   63.78%   +<.01%     
==========================================
  Files          38       38              
  Lines        1778     1814      +36     
==========================================
+ Hits         1134     1157      +23     
- Misses        593      604      +11     
- Partials       51       53       +2
Impacted Files Coverage Δ
server/events/apply_executor.go 0% <0%> (ø) ⬆️
server/events/project_pre_execute.go 96.61% <100%> (+0.53%) ⬆️
server/events/plan_executor.go 89.47% <100%> (+1.47%) ⬆️
server/events/event_parser.go 97.25% <89.47%> (-1.01%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cf4fd37...3ec956e. Read the comment docs.

@lkysow lkysow force-pushed the comment-syntax branch 2 times, most recently from 2b62a63 to 807446c Compare February 26, 2018 23:09
@lkysow lkysow merged commit ee48039 into master Feb 26, 2018
@lkysow lkysow deleted the comment-syntax branch February 26, 2018 23:15
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.

1 participant