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

Ignore .sshimageBuild and .bgsyncimageBuild directories by default, fixes #1762 #1763

Merged

Conversation

rmcveigh
Copy link
Contributor

@rmcveigh rmcveigh commented Aug 3, 2019

The Problem/Issue/Bug:

Issue #1762
.sshimageBuild and .bgsyncimageBuild are not ignored by default after performing a ddev config

How this PR Solves The Problem:

Adjusts the pkg/ddevapp/config.go file to ignore the directories

Manual Testing Instructions:

ddev config should add .sshimagebuild and .bgsyncimagebuild to the .ddev/.gitignore file

Automated Testing Overview:

Tests were not added due to the simplicity of the change.

Related Issue Link(s):

Issue: #1762

Release/Deployment notes:

A new ddev config run should add the .s to the filenames in the .gitignore file in .ddev if the file is marked to be auto generated/altered.

@CLAassistant
Copy link

CLAassistant commented Aug 3, 2019

CLA assistant check
All committers have signed the CLA.

@rmcveigh rmcveigh changed the title rmcveigh: ignore sshimagebuild and bgsyncimagebuild by default Ignore sshimagebuild and bgsyncimagebuild by default Aug 3, 2019
@rmcveigh rmcveigh changed the title Ignore sshimagebuild and bgsyncimagebuild by default Ignore .sshimageBuild and .bgsyncimageBuild directories by default Aug 3, 2019
Copy link
Member

@rfay rfay left a comment

Choose a reason for hiding this comment

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

This is awesome, thanks so much for catching and fixing this!

@rfay rfay changed the title Ignore .sshimageBuild and .bgsyncimageBuild directories by default Ignore .sshimageBuild and .bgsyncimageBuild directories by default, fixes #1762 Aug 4, 2019
@rfay rfay merged commit 915a9d0 into ddev:master Aug 4, 2019
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.

None yet

3 participants