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

modifies lib exclude, adds example lib include #4563

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

Conversation

mark-lazarides
Copy link

explicitly excludes the _Libraries directory, and provides example of adding sub directories back in. Should be used for including custom and third party (non -Beckhoff shipped) libraries, improving maintainability

Reasons for making this change:
The _Libraries directory was blanket ignored before. If using a project with non-Beckhoff libraries, it can become hard to maintain the project, as sourcing those libraries is challenging. The ignore file was modified to explicitly ignore the _Libraries directory (adding wildcarding to ensure it is expllicitly selected).

A commented out example is provided guiding users on how to include specific library directories.

explicitly excludes the `_Libraries` directory, and provides example of
adding sub directories back in. Should be used for including custom and
third party (non -Beckhoff shipped) libraries, improving maintainability
@mark-lazarides
Copy link
Author

@sagatowski - as original submitter of the TwinCAT3.gitignore, could you comment on this PR?

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