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

Update gitignore with additional Copado elements #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

delokoni
Copy link

To avoid deployment errors due to Copado Managed Package elements, more metadata types were added to the gitignore file.

To avoid deployment errors due to Copado Managed Package elements, more metadata types were added to the gitignore file.
workflows/copado__*
objects/copado__*
objectTranslations/copado__*
layouts/copado__*
Copy link

@drcopa drcopa Nov 15, 2017

Choose a reason for hiding this comment

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

Some customers personalize Copado in Sandboxes and then deploy the changes using Copado. I originally included some of these lines in the gitignore but customers opened cases because Copado changes in objects/layouts did not appear when committing them to Git. To avoid these type of cases we only included metadata types in the gitignore that cannot be modified in Managed Packages (apex classes, etc).
I would do the same in your changes, only leave the ones that cannot be modified.

Copy link
Author

Choose a reason for hiding this comment

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

Sounds fair.
Maybe include the rest commented with a note

Copy link

Choose a reason for hiding this comment

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

Yeah that would be good!

Choose a reason for hiding this comment

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

what about something like:

classes/*__*
triggers/*__*
pages/*__*
components/*__*

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.

3 participants