Skip to content

[BUG] Fix nargin check - #3

Open
agahkarakuzu wants to merge 6 commits into
JosePMarques:masterfrom
agahkarakuzu:master
Open

agahkarakuzu wants to merge 6 commits into
JosePMarques:masterfrom
agahkarakuzu:master

Conversation

@agahkarakuzu

Copy link
Copy Markdown
  • I noticed a small bug in T1B1correctpackageTFL.m regarding argument parsing, corrected in d53a92b.
  • Added data folder to .gitignore
  • Did some minor syntax fixes whenever I saw them.

@mathieuboudreau

Copy link
Copy Markdown

@agahkarakuzu, I'm quite certain that @JosePMarques doesn't want to ignore the data folder, as it is key for his examples. Since this is mostly a static repo, the data was uploaded once and will never change again, so something like OSF would be overkill for here.

@agahkarakuzu

agahkarakuzu commented Oct 9, 2019 •

Copy link
Copy Markdown
Author

Since this is mostly a static repo, the data was uploaded once and will never change again

Sounds like another good reason to ignore tracking data folder?

I just did it for the sake of good practice, happy to revert at @JosePMarques's discretion.

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